-
Notifications
You must be signed in to change notification settings - Fork 180
RUST-950 Enable connection to a load balancer #415
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.
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
Merged
abr-egn
merged 29 commits into
mongodb:master
from
abr-egn:RUST-950/load-balancer-connection
Aug 11, 2021
Merged
Changes from all commits
Commits
Show all changes
29 commits
Select commit
Hold shift + click to select a range
3f1c162
add a non-public load balancer feature flag
abr-egn de0deab
recognize loadBalanced URL param
abr-egn 797f488
parse TXT option
abr-egn 1fe9d68
validate options
abr-egn 4728d7f
introduce topology and server types
abr-egn 10537a3
mark connection pool ready
abr-egn 737b414
create initial server with LoadBalancer type
abr-egn 83f0d8d
send loadBalanced hello parameter, check for serviceId in reply
abr-egn bb19694
include service_id in TestIsMasterCommandResponse
abr-egn 54916f3
add generation map
abr-egn 45b7963
determine connection generation from service id, and use such in stal…
abr-egn 44bba33
track generation and connection count
abr-egn c47e59a
fix tests
abr-egn 31fa505
rustfmt
abr-egn b6d0b2e
clippy
abr-egn 2ee6e39
remodel generation
abr-egn f5630db
rustfmt
abr-egn 1c8c130
lint
abr-egn bd6fe46
distinguish pre-hello failures from post-hello
abr-egn 8d8a54f
fix sdam test
abr-egn b3c4ad1
rustfmt
abr-egn 518896e
fix TXT option case match
abr-egn fb6c86d
panic in test cfg on load balanced mismatch
abr-egn 334a463
fix is_before_completion
abr-egn d21165b
tidy macro
abr-egn 4c38e7a
rustfmt and clippy
abr-egn 514328c
another case fix
abr-egn 26203f4
review updats
abr-egn 78bfb11
reuse is_load_balanced
abr-egn File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This awkward double-flag construction allows
load_balanced
to be populated by the URI/TXT parsing as it will be on release, but prevents that from being used outside of tests that have access to setallow_load_balanced
to be true.