Conversation
CRZbulabula
commented
Apr 13, 2026
- For ClusterPartitionFetcher, we should remove the duplicated consensus group ids before fetching schema partitions.
- The previous statistics maps should be cleaned when the corresponding ConfigNode is no longer the leader; otherwise the ConfigNode would take obsolete statistics when next time it becomes the leader.
There was a problem hiding this comment.
Pull request overview
This PR improves load-balancing and partition-fetching correctness in IoTDB’s cluster mode by reducing redundant region-route lookups and preventing stale load statistics from being reused across ConfigNode leadership changes.
Changes:
- Deduplicate consensus group IDs when resolving schema partitions in
ClusterPartitionFetcher. - Clear previously recorded node/region/consensus statistics when stopping
EventService(e.g., on leadership change).
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| iotdb-core/datanode/src/main/java/org/apache/iotdb/db/queryengine/plan/analyze/ClusterPartitionFetcher.java | Attempts to deduplicate schema consensus group IDs before fetching replica sets. |
| iotdb-core/confignode/src/main/java/org/apache/iotdb/confignode/manager/load/service/EventService.java | Clears cached “previous” statistics maps on event service stop to avoid stale state after leadership changes. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
...node/src/main/java/org/apache/iotdb/db/queryengine/plan/analyze/ClusterPartitionFetcher.java
Show resolved
Hide resolved
.../confignode/src/main/java/org/apache/iotdb/confignode/manager/load/service/EventService.java
Outdated
Show resolved
Hide resolved
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
|
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #17463 +/- ##
============================================
- Coverage 39.82% 39.82% -0.01%
Complexity 312 312
============================================
Files 5135 5135
Lines 347073 347083 +10
Branches 44220 44222 +2
============================================
- Hits 138221 138215 -6
- Misses 208852 208868 +16 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|


