Uh oh!
There was an error while loading. Please reload this page.
branch-4.1: [fix](group commit) support forward group commit stream load (#63594) - #65350
Conversation
hello-stephen
commented
Jul 8, 2026
Thank you for your contribution to Apache Doris. Please clearly describe your PR:
|
There was a problem hiding this comment.
Pull request overview
Enables an optional FE-side routing path where group-commit stream load requests received by the master FE can be forwarded to a follower FE (round-robin), improving support for multi-follower deployments (especially in cloud follower mode). Adds a regression test and a new FE config flag to control the behavior.
Changes:
- Add master-side forwarding logic for group-commit stream load PUT requests to follower FEs with a pre-check fallback.
- Introduce
enable_forward_group_commit_stream_load_to_followerFE config (master-only, mutable) to toggle forwarding. - Add a regression test validating group-commit stream loads succeed in a multi-follower setup when forwarding is enabled.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| regression-test/suites/load_p0/stream_load/test_group_commit_stream_load_multi_follower.groovy | Adds a docker-based multi-FE follower-mode regression test for group-commit stream load. |
| fe/fe-core/src/main/java/org/apache/doris/service/FrontendServiceImpl.java | Implements forwarding of group-commit stream load PUT from master to follower FE (round-robin) with pre-check and fallback. |
| fe/fe-common/src/main/java/org/apache/doris/common/Config.java | Adds a new config flag to enable/disable forwarding behavior. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Uh oh!
There was an error while loading. Please reload this page.
mymeiyi
commented
Jul 8, 2026
run buildall |
hello-stephen
commented
Jul 8, 2026
FE Regression Coverage ReportIncrement line coverage |
Uh oh!
There was an error while loading. Please reload this page.
pick #63594