[test] Tolerate removed tablet servers in FlussClusterExtension wait helpers
- Dominant language
- Java
- Stars
- 2.1k
- Forks
- 625
- Avg merge
- 3d 14h
- Merged PRs (30d)
- 97
Description
### Search before asking
- [x] I searched in the [issues](https://github.com/apache/fluss/issues) and found nothing similar.
### Description
Follow-up to #3667: the same NPE race exists at three more getTabletServerById call sites in FlussClusterExtension. During a failover the lookup can return null (leader temporarily NO_LEADER, or the server already removed), and since retry(...) only retries AssertionError, the NPE fails the test instead of letting the wait continue.
### Willingness to contribute
- [x] I'm willing to submit a PR!
Contributor guide
No contributing guide indexed for this repository
Research direction
Start in FlussClusterExtension and locate the three getTabletServerById call sites described in the issue; compare their retry(...) usage with the fix discussed in #3667. Verify that a null lookup during failover does not cause an NPE and that the wait continues until the expected condition is met.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- java
- Domain
- testing
- Issue type
- Bug
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 72/100