wenzhenghu opened a new pull request, #65699:
URL: https://github.com/apache/doris/pull/65699
### What problem does this PR solve?
Issue Number: None
Related PR: HYDCP/hy-doris#68
Problem Summary:
Port the configurable BE shutdown tablet sweep from branch-3.1 to master.
This change adds dynamic round budget and interval configs, bounds each
shutdown-tablet fetch and scan pass, preserves retry ordering for failed
tablets, and exposes backlog, per-round outcome, and timing metrics. Follow-up
review fixes tighten the dynamic config bounds to [1, 10000] and [0, 10000],
clarify reached-end round termination semantics, extend BE unit coverage for
boundary and retry-order cases, and rename the success counters and metrics
from moved to resolved so the observability semantics match the implementation.
### Release note
Support configuring BE shutdown tablet sweep pacing and observability on
master.
### Check List (For Author)
- Test <!-- At least one of them must be included. -->
- [ ] Regression test
- [ ] Unit Test
- [ ] Manual test (add detailed scripts or steps below)
- [x] 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.
- [x] Other reason <!-- Add your reason? -->
- Ran `./build-support/clang-format.sh` and `git diff --check`
during migration and follow-up review fixes.
- Attempted targeted `./run-be-ut.sh --run
--filter=TabletMgrTest...`, but the local build is currently blocked by an
existing `contrib/openblas/getarch_2nd.c` compile failure before reaching the
target cases.
- The original branch-3.1 implementation was previously
validated in HYDCP/hy-doris#68; this PR is a clean master migration plus
review-driven fixes.
- Behavior changed:
- [ ] No.
- [x] Yes. Shutdown tablet cleanup pacing is now dynamically
configurable, and shutdown sweep metrics and unit coverage are added on master.
- 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]
