Skip to content

[CI] Wait for table metadata in authorization test - #3875

Merged
fresh-borzoni merged 2 commits into
apache:mainfrom
litiliu:codex/fluss-3747-wait-table-metadata
Aug 12, 2026
Merged

[CI] Wait for table metadata in authorization test#3875
fresh-borzoni merged 2 commits into
apache:mainfrom
litiliu:codex/fluss-3747-wait-table-metadata

Conversation

@litiliu

@litiliu litiliu commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Purpose

Linked issue: close #3747

FlussAuthorizationITCase.testDescribeTableOperation can call listOffsets after the bucket replica is online but before the TabletServer metadata cache contains the table ID. The authorization check then fails with UnknownTableOrBucketException instead of exercising the intended ACL behavior.

Brief change log

  • Establish bucket 0 replica and TabletServer metadata readiness after creating the default table in the shared test setup.
  • Remove duplicate readiness waits from the affected tests.

Tests

  • ./mvnw -pl fluss-client -am -Dtest=FlussAuthorizationITCase -Dsurefire.failIfNoSpecifiedTests=false test
  • git diff --check

API and Format

No API or storage format changes.

Documentation

No documentation changes.

Wait until the relevant table bucket replicas and TabletServer metadata caches are ready before exercising listOffsets authorization.
@litiliu
litiliu marked this pull request as ready for review August 6, 2026 04:21
@litiliu litiliu changed the title [test] Wait for table metadata in authorization test [CI] Wait for table metadata in authorization test Aug 6, 2026

@fresh-borzoni fresh-borzoni left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@litiliu Thank you for the PR, LGTM overall, left a suggestion, PTAL

@litiliu
litiliu requested a review from fresh-borzoni August 11, 2026 02:53

@fresh-borzoni fresh-borzoni left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@litiliu Thank you, LGTM 👍

@fresh-borzoni
fresh-borzoni merged commit 6e38147 into apache:main Aug 12, 2026
9 checks passed
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 FlussAuthorizationITCase.testDescribeTableOperation

2 participants