Only send table deletion to related region groups#17896
Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #17896 +/- ##
============================================
+ Coverage 40.54% 40.83% +0.28%
+ Complexity 2622 318 -2304
============================================
Files 5244 5245 +1
Lines 362367 363213 +846
Branches 46651 46783 +132
============================================
+ Hits 146938 148333 +1395
+ Misses 215429 214880 -549 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Caideyipi
left a comment
There was a problem hiding this comment.
LGTM. I reviewed the change and did not find any blocking correctness issues. Routing table deletions through getRegionGroupsByTime by database and deletion time range matches the ConfigNode partition lookup semantics, and an empty region result is handled as a no-op by the write dispatch path. Optional follow-up: add coverage for non-success and empty/unset regionReplicaSets responses from ConfigNode.
|



Description
Verification