bobhan1 opened a new pull request, #66727:
URL: https://github.com/apache/doris/pull/66727
### What problem does this PR solve?
Issue Number: None
Related PR: #56061
Problem Summary: `MetaServiceHttpTest.ShortGetTabletStatsDebugStringTest`
enables the mutable global config `enable_idempotent_request_injection` but
does not restore it. Cloud Meta Service tests run in the same process, so later
tests inherit chaos injection and unexpectedly schedule delayed request
replays. In an ASan run, a later `UpdateTabletRequest` replay executed after
its test-owned service proxy had been destroyed and caused the test binary to
abort. This change saves the previous config value and restores it in the
existing deferred cleanup, isolating the test from subsequent cases.
### Release note
None
### Check List (For Author)
- Test
- [x] Unit Test
- `./run-cloud-ut.sh --run
--filter=meta_service_test:MetaServiceHttpTest.ShortGetTabletStatsDebugStringTest
-j100`
- `./run_all_tests.sh --test meta_service_test --filter
'MetaServiceHttpTest.ShortGetTabletStatsDebugStringTest:MetaServiceOperationLogTest.UpdateVersionedTabletMeta'`
- `build-support/check-format.sh`
- `git diff --check`
- Behavior changed:
- [x] No. This only restores test configuration after the test finishes.
- Does this need documentation?
- [x] No.
### Check List (For Reviewer who merge this PR)
- [ ] Confirm the release note
- [ ] Confirm test cases
- [ ] Confirm document
- [ ] Add branch pick label
--
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]
