Upgrade the version from 8.0.3 to 9.0.2#7
Merged
Merged
Conversation
* First draft for the smart feature * TopologyAwareDataSource Code * Adding Refresh Interval + Changes in Fallback toology * Fallback tests + minor improvements * Review comments + Multiple host in URL * Minor changes to refresh * Review comments * Improved the tests + updated the README * MultiThreading bug fix * Review Comments * Fallback Test cases + fallback topology fix * Change namespace from Npgsql to YBNpgsql * Changing the version + Assembly name * Renaming namespace in new files * Renaming namespace in few more files * Preparing for release * Fix for the fallback issue * Resolved some build issues * Added the flags for supression of sqls * Change support of Unlisten and Advisory Locks to false * Changed the method for retrieving of allowed placement values * Removed local package source from Nuget.config + Added the new connection properties to Readme --------- Co-authored-by: Sfurti Sarah <ssarah@Aadhas-MacBook-Pro.local> Co-authored-by: Amogh Shetkar <ashetkar@yugabyte.com>
* Changed PackageIDs and AssemblyName for Nuget publish
…es (#6) * Initial changes to ClusterAwareLoadBalancer for RR support * Initial changes for RR support * Test cleanup * Addressed the review comments * Modified tests to include client side verification + Check if connections are closed properly * Make GetLoad static * Changes as per review comments * Refactored the tests to add method call in place of code repititions * Format the new code
…ture for hasbetternodeavailable method
…er + Fix for better node availability
ashetkar
reviewed
Sep 23, 2025
Collaborator
|
Please update the README with details on the smart feature. You can refer to the READMEs of other drivers (pgjdbc/pgx/psycopg2). |
Collaborator
Author
The Readme already has load balance features updated |
ashetkar
approved these changes
Sep 24, 2025
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Changes made:
Rebased the smart driver changes on top of v9.0.2 of upstream driver
Fixed issues with fallback to upstream behaviour as well as better node available behaviour as a result of the previous change.
Tests done:
Uniform load balance tests (
LoadBalanceTests.cs)Topology Aware Tests (
FallBackTopologyTests.cs)RR support Tests
Steps followed: https://docs.google.com/document/d/11HXDKzKjl4f2cduPK8IfUfedpfDoLiKiSQpPtjTD2ek/edit?usp=sharing