Yukang-Lian opened a new pull request, #67592:
URL: https://github.com/apache/doris/pull/67592
### What problem does this PR solve?
Issue Number: None
Related PR: #66585
Problem Summary: Backport #66585 to `branch-3.1`.
Group Commit asks `CloudSystemInfoService` for the backend pool selected by
a physical or virtual compute group, but then compared those physical backends
with the original virtual name. That rejected every healthy candidate and could
also reuse a cached backend after it left the current pool.
This backport keeps virtual compute group routing inside
`CloudSystemInfoService`: cache hits use a scoped lookup in the current
selected pool, while random selection trusts the already scoped candidate pool
and performs only health checks. The implementation is adapted to the
branch-3.1 Group Commit and cloud-system APIs.
### Release note
Fix Group Commit loads routed through virtual compute groups.
### Check List (For Author)
- Test
- [ ] Regression test
- [x] Unit Test
- Focused tests are included; hosted CI is requested with `run
buildall`.
- `build-support/check-format.sh`: passed
- `git diff --check`: passed
- [ ] Manual test
- [ ] No need to test or manual test
- Behavior changed: Yes. Group Commit now accepts healthy backends returned
for a virtual compute group and invalidates cached backends outside its current
selected pool.
- Does this need documentation? No
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
