Skip to content

MINOR: Deflake Streams position restart integration test - #23111

Merged
bbejeck merged 3 commits into
apache:trunkfrom
ezhou413:ezhou/deflake-position-restart-test
Aug 11, 2026
Merged

MINOR: Deflake Streams position restart integration test#23111
bbejeck merged 3 commits into
apache:trunkfrom
ezhou413:ezhou/deflake-position-restart-test

Conversation

@ezhou413

@ezhou413 ezhou413 commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

This PR deflakes TransactionalPositionRestartIntegrationTest by moving
shared state off static variables in PositionRestartIntegrationTest

https://develocity.apache.org/scans/tests?search.rootProjectNames=kafka&search.timeZoneId=America%2FChicago&tests.container=org.apache.kafka.streams.integration.TransactionalPositionRestartIntegrationTest

Reviewers: Bill Bejeck bbejeck@apache.org

@bbejeck bbejeck added streams tests Test fixes (including flaky tests) ci-approved labels Aug 6, 2026
@github-actions github-actions Bot added the small Small PRs label Aug 11, 2026

@bbejeck bbejeck 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.

Thanks for the fix @ezhou413 LGTM

@bbejeck
bbejeck merged commit d4bc6cd into apache:trunk Aug 11, 2026
23 checks passed
@bbejeck

bbejeck commented Aug 11, 2026

Copy link
Copy Markdown
Member

Merged #23111 into trunk

bbejeck pushed a commit that referenced this pull request Aug 11, 2026
This PR deflakes TransactionalPositionRestartIntegrationTest by moving
shared state off static variables in PositionRestartIntegrationTest


https://develocity.apache.org/scans/tests?search.rootProjectNames=kafka&search.timeZoneId=America%2FChicago&tests.container=org.apache.kafka.streams.integration.TransactionalPositionRestartIntegrationTest

Reviewers: Bill Bejeck <bbejeck@apache.org>
@bbejeck

bbejeck commented Aug 11, 2026

Copy link
Copy Markdown
Member

cherry-picked to 4.4

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

Labels

ci-approved small Small PRs streams tests Test fixes (including flaky tests)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants