Skip to content

[server] Fix flaky RebalanceManagerITCase.testRebalanceWithRemoteLog#3313

Open
Kaixuan-Duan wants to merge 1 commit into
apache:mainfrom
Kaixuan-Duan:issue-2974-flaky-RebalanceManagerITCase
Open

[server] Fix flaky RebalanceManagerITCase.testRebalanceWithRemoteLog#3313
Kaixuan-Duan wants to merge 1 commit into
apache:mainfrom
Kaixuan-Duan:issue-2974-flaky-RebalanceManagerITCase

Conversation

@Kaixuan-Duan
Copy link
Copy Markdown
Contributor

Purpose

Linked issue: close #2974

Brief change log

  • Use retry to wait until the in-memory manifest is ready before reading remoteLogSize, since waitUntilSomeLogSegmentsCopyToRemote only ensures ZK has the manifest handle but the in-memory RemoteLogTablet may not be updated yet by LogTieringTask.
  • Change the assertion on remote log segment count after rebalance from isEqualTo to isGreaterThanOrEqualTo, since remoteLogSize is captured when async remote log copy may not have finished uploading all segments yet, so the new leader's manifest can contain more segments.

Tests

./mvnw -pl fluss-server -Dtest=RebalanceManagerITCase#testRebalanceWithRemoteLog -DfailIfNoTests=false verify

API and Format

Documentation

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[test] Unstable test RebalanceManagerITCase.testRebalanceWithRemoteLog

1 participant