Skip to content

KAFKA-20417: Add KIP-1035 self-managed offset lifecycle integration tests#21999

Open
bbejeck wants to merge 2 commits intoapache:trunkfrom
bbejeck:KAFKA-20417-self-managed-offset-lifecycle-tests
Open

KAFKA-20417: Add KIP-1035 self-managed offset lifecycle integration tests#21999
bbejeck wants to merge 2 commits intoapache:trunkfrom
bbejeck:KAFKA-20417-self-managed-offset-lifecycle-tests

Conversation

@bbejeck
Copy link
Copy Markdown
Member

@bbejeck bbejeck commented Apr 8, 2026

Adds SelfManagedOffsetLifecycleIntegrationTest — companion to the
recovery tests.

Validates the happy path for column-family-based offset management.
After each clean shutdown, tests directly inspect the CF to assert
status=closed and offsets populated. Covers:

  • State preservation across clean restarts - Multiple
    restart cycles - No changelog restoration after clean shutdown (via
    StateRestoreListener) - Clean restart with no data processed
  • Interactive query state verification across restarts

All tests parameterized for ALOS and EOS.

Reviewers: Matthias J. Sax matthias@confluent.io

@github-actions github-actions bot added streams tests Test fixes (including flaky tests) labels Apr 8, 2026
@bbejeck bbejeck force-pushed the KAFKA-20417-self-managed-offset-lifecycle-tests branch from c1f15d3 to a9ad811 Compare April 10, 2026 19:18
@bbejeck
Copy link
Copy Markdown
Member Author

bbejeck commented Apr 10, 2026

@mjsax rebased PR

// After clean shutdown with no data, status should still be closed
assertStoreStatus(0L);

// Restart — should not treat empty CF as corruption
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.

How do we verify this?

Copy link
Copy Markdown
Member

@mjsax mjsax left a comment

Choose a reason for hiding this comment

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

Overall LGTM. Just one question.

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

Labels

streams tests Test fixes (including flaky tests)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants