Skip to content

[test] Fix flaky FlussAdminITCase.testDynamicConfigs#3303

Open
Kaixuan-Duan wants to merge 1 commit into
apache:mainfrom
Kaixuan-Duan:issue-3127-unstable-dynamic-configs-test
Open

[test] Fix flaky FlussAdminITCase.testDynamicConfigs#3303
Kaixuan-Duan wants to merge 1 commit into
apache:mainfrom
Kaixuan-Duan:issue-3127-unstable-dynamic-configs-test

Conversation

@Kaixuan-Duan
Copy link
Copy Markdown
Contributor

@Kaixuan-Duan Kaixuan-Duan commented May 12, 2026

Purpose

Linked issue: close #3127

Brief change log

  • Modify FlussAdminITCase.testDynamicConfigs test method
  • Add a verify step that first checks cluster config has been updated before checking table info
  • Use describeClusterConfigs to confirm cluster config is updated first, then verify table info reflects latest config
  • Avoid the race condition where LakeCatalogDynamicLoader hasn't finished reconfiguring when table info is queried

Tests

  • FlussAdminITCase#testDynamicConfigs test passed and runs stably
  • Run multiple times to verify test stability

./mvnw -pl fluss-client -Dtest='FlussAdminITCase#testDynamicConfigs' -DfailIfNoTests=false test

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 FlussAdminITCase.testDynamicConfigs

1 participant