apache / apache/fluss

[test] Tolerate removed tablet servers in FlussClusterExtension wait helpers

Open Beginner friendly
#3,716 1 comment 0 reactions 0 assignees View on GitHub
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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.