shuke987 opened a new pull request, #65577:
URL: https://github.com/apache/doris/pull/65577
### What problem does this PR solve?
Issue Number: None
Related PR: #62805
Problem Summary:
`FrontendServiceImplTest` starts a non-cloud mocked FE and BE for its shared
fixture. However,
`testGetTabletReplicaInfosNullJobReturnsCancelledWithoutNpe` temporarily sets
`Config.cloud_unique_id`, which switches the process-wide environment to
cloud
mode.
If the background heartbeat runs during that window, it enters the cloud
heartbeat path with a non-cloud `SystemInfoService`. The resulting failed
heartbeat marks the shared mock BE not alive, so later table-creation tests
intermittently fail with zero available backends.
Move this cloud-specific test to `FrontendServiceImplCloudTest`, which has no
`MockedFrontend` or mock BE fixture. The original `getTabletReplicaInfos`
request and `CANCELLED` assertions remain unchanged, so test coverage is
preserved while the global cloud-mode change is isolated from the background
heartbeat.
### Release note
None
### Check List (For Author)
- Test
- [ ] 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
Validation on latest `master`:
- `./run-fe-ut.sh --run
org.apache.doris.service.FrontendServiceImplCloudTest`: 1 test passed
- `./run-fe-ut.sh --run org.apache.doris.service.FrontendServiceImplTest`:
18 tests passed
- Behavior changed:
- [x] No.
- [ ] Yes.
- Does this need documentation?
- [x] No.
- [ ] Yes.
### 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]
