impl(bigtable): add option to select directpath initialization mode - #16391
Conversation
There was a problem hiding this comment.
Code Review
This pull request introduces DirectPathInitializationModeOption to allow configuring the initialization behavior of DirectPath in Google Cloud Bigtable, supporting both blocking and speculative asynchronous modes. The feedback suggests refactoring MakeBlockingDirectPathDataConnection to eliminate duplicated StubManager creation logic using a helper lambda, and optimizing MakeDirectPathDataConnection to avoid unnecessary copies of components.options while preventing potential order-of-evaluation issues.
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #16391 +/- ##
==========================================
+ Coverage 92.24% 92.26% +0.01%
==========================================
Files 2246 2246
Lines 212035 212121 +86
==========================================
+ Hits 195598 195710 +112
+ Misses 16437 16411 -26 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
No description provided.