deardeng opened a new pull request, #65611:
URL: https://github.com/apache/doris/pull/65611
Adds the FE-side topology metadata only. No scheduling, replica selection or
load behavior changes.
A FE can now declare which resource group (BE `tag.location`) it sits in, so
that operators and later features can reason about cross-AZ / cross-IDC
deployments:
- `local_resource_group` FE config, overridable by the
`--local_resource_group` start_fe.sh option or the `DORIS_LOCAL_RESOURCE_GROUP`
environment variable. Precedence is command line > environment > fe.conf. The
value must be a valid `tag.location` name, otherwise the FE refuses to start.
- The value is reported through the FE heartbeat
(TFrontendPingFrontendResult) and kept on Frontend as runtime-only state, so
the master can see every FE's group.
- Exposed as a new `LocalResourceGroup` column, appended at the end of `SHOW
FRONTENDS` and `frontends()` so existing column positions do not move.
Because the command line value has to override fe.conf, it can only be
resolved once Config is loaded. DorisFE.parseArgs() is therefore split into
parsing (parseArgs) and mode selection (buildCommandLineOptions). Both halves
are still called at exactly the point where parseArgs() used to be, so the
version / helper / image modes keep returning before the recovery and
drop-backends system properties are applied, and those flags remain ignored in
those modes as before. Only resolveLocalResourceGroup() runs after
Config.init().
### What problem does this PR solve?
Issue Number: close #xxx
Related PR: #xxx
Problem Summary:
### Release note
None
### Check List (For Author)
- Test <!-- At least one of them must be included. -->
- [ ] Regression test
- [x] Unit Test
- [ ] Manual test (add detailed scripts or steps below)
- [ ] No need to test or manual test. Explain why:
- [ ] This is a refactor/code format and no logic has been changed.
- [ ] Previous test can cover this change.
- [ ] No code files have been changed.
- [ ] Other reason <!-- Add your reason? -->
- Behavior changed:
- [x] No.
- [ ] Yes. <!-- Explain the behavior change -->
- Does this need documentation?
- [x] No.
- [ ] Yes. <!-- Add document PR link here. eg:
https://github.com/apache/doris-website/pull/1214 -->
### Check List (For Reviewer who merge this PR)
- [ ] Confirm the release note
- [ ] Confirm test cases
- [ ] Confirm document
- [ ] Add branch pick label <!-- Add branch pick label that this PR should
merge into -->
--
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]
