Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Force connection reuse #47

Merged
merged 14 commits into from
Feb 7, 2024
Merged

Force connection reuse #47

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
5bcb4f9
Update crates
Jake-Shadle Feb 7, 2024
05ade46
Add repro for connections
Jake-Shadle Feb 7, 2024
61fe0e3
Do async as well
Jake-Shadle Feb 7, 2024
85439a1
Add test for async, make it work on CI
Jake-Shadle Feb 7, 2024
af61a1f
Scale flex to proc count, eg on my 24 core machine it fluctuates betw…
Jake-Shadle Feb 7, 2024
761da3f
Add test on CI
Jake-Shadle Feb 7, 2024
f6e2e04
Fix lints
Jake-Shadle Feb 7, 2024
9826221
Only run once
Jake-Shadle Feb 7, 2024
4481f13
Force threads
Jake-Shadle Feb 7, 2024
16ac605
Silence CI warnings
Jake-Shadle Feb 7, 2024
b969d1e
Special case CI
Jake-Shadle Feb 7, 2024
c0c8998
Check old version, hopefully CI fails
Jake-Shadle Feb 7, 2024
74902dc
Restore fix
Jake-Shadle Feb 7, 2024
3d07dc1
Update CHANGELOG
Jake-Shadle Feb 7, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 8 additions & 4 deletions .github/workflows/rust-ci.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ jobs:
name: Lint
runs-on: ubuntu-22.04
steps:
- uses: actions/checkout@v3
- uses: actions/checkout@v4
- uses: dtolnay/rust-toolchain@stable
with:
components: "clippy, rustfmt"
Expand All @@ -37,26 +37,30 @@ jobs:
features: ["--features git", "--features sparse", "--features local-builder,sparse"]
runs-on: ${{ matrix.os }}
steps:
- uses: actions/checkout@v3
- uses: actions/checkout@v4
- uses: dtolnay/rust-toolchain@stable
- run: cargo fetch
- name: cargo test build
run: cargo build --tests ${{ matrix.features }}
- name: cargo test
run: cargo test ${{ matrix.features }}
- name: Test connection reuse
if: ${{ matrix.features == '--features sparse' }}
run: |
cargo run --manifest-path tests/connect/Cargo.toml

deny-check:
name: cargo-deny
runs-on: ubuntu-22.04
steps:
- uses: actions/checkout@v3
- uses: actions/checkout@v4
- uses: EmbarkStudios/cargo-deny-action@v1

publish-check:
name: Publish Check
runs-on: ubuntu-22.04
steps:
- uses: actions/checkout@v3
- uses: actions/checkout@v4
- uses: dtolnay/rust-toolchain@stable
- run: cargo fetch
- name: cargo publish check
Expand Down
2 changes: 1 addition & 1 deletion .gitignore
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
/target
**/target
**/*.rs.bk
Cargo.lock
tests/flock/target
3 changes: 3 additions & 0 deletions CHANGELOG.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,9 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0

<!-- next-header -->
## [Unreleased] - ReleaseDate
### Fixed
- [PR#47](https://github.com/EmbarkStudios/tame-index/pull/47) fixed [#46](https://github.com/EmbarkStudios/tame-index/issues/46) by ensuring one full DNS lookup and request response roundtrip is made before going wide to ensure that excessive DNS lookups and connections are not made.

## [0.9.2] - 2024-01-21
### Changed
- [PR#45](https://github.com/EmbarkStudios/tame-index/pull/45) bumped `gix` -> 0.58
Expand Down
Loading