Not relying on random function to select least loaded server by concurrent
threads. Instead putting a small synch block to just quickly determine the least,
optimistically increment the connection counter assuming connection will
be successful. However if connection fails then decrementing the
reference count or removing it from the internal 'host->conn' map
All Submissions:
[✅] Have you followed the guidelines in our Contributing document?
[✅] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
New Feature Submissions:
[✅] Does your submission pass tests?
[ ] Does mvn checkstyle:check pass ?
[ ] Have you added your new test classes to an existing test suite? - Tested with existing yb_sample_apps Sqlnserts workload
Changes to Existing Features:
[ ] Does this break existing behaviour? If so please explain.
[✅] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your core changes, as applicable?
[✅] Have you successfully run tests with your changes locally?
Not relying on random function to select least loaded server by concurrent threads. Instead putting a small synch block to just quickly determine the least, optimistically increment the connection counter assuming connection will be successful. However if connection fails then decrementing the reference count or removing it from the internal 'host->conn' map
All Submissions:
New Feature Submissions:
Changes to Existing Features: