Skip to content

MINOR: Speed up more streams restart tests - #23271

Closed
Russole wants to merge 1 commit into
apache:trunkfrom
Russole:minor/speed-up-more-streams-restart-tests
Closed

MINOR: Speed up more streams restart tests#23271
Russole wants to merge 1 commit into
apache:trunkfrom
Russole:minor/speed-up-more-streams-restart-tests

Conversation

@Russole

@Russole Russole commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

Summary

Speed up several Streams restart-based integration tests by leaving the
group before immediate restarts with the same application.id.

Testing

Ran affected integration test methods:

./gradlew :streams:integration-tests:integrationTest \
  --tests
org.apache.kafka.streams.integration.SelfJoinUpgradeIntegrationTest.shouldUpgradeWithTopologyOptimizationOff
\
  --tests
org.apache.kafka.streams.integration.SelfJoinUpgradeIntegrationTest.shouldRestartWithTopologyOptimizationOn
\
  --tests
org.apache.kafka.streams.integration.SessionsStoreWithHeadersTest.shouldRestore
\
  --tests
org.apache.kafka.streams.integration.SessionsStoreWithHeadersTest.shouldManualUpgradeFromTimestampedToHeaders
\
  --tests
org.apache.kafka.streams.integration.TimestampedKeyValueStoreWithHeadersTest.shouldRestore
\
  --tests
org.apache.kafka.streams.integration.TimestampedKeyValueStoreWithHeadersTest.shouldManualUpgradeFromTimestampedToHeaders
\
  --tests
org.apache.kafka.streams.integration.TimestampedWindowStoreWithHeadersTest.shouldRestore
\
  --tests
org.apache.kafka.streams.integration.TimestampedWindowStoreWithHeadersTest.shouldManualUpgradeFromTimestampedToHeaders
\
  --tests
org.apache.kafka.streams.integration.VersionedKeyValueStoreIntegrationTest.shouldRestore
\
  --tests
org.apache.kafka.streams.integration.VersionedKeyValueStoreIntegrationTest.shouldManualUpgradeFromNonVersionedTimestampedToVersioned
\
  --tests
org.apache.kafka.streams.integration.VersionedKeyValueStoreIntegrationTest.shouldManualUpgradeFromNonVersionedNonTimestampedToVersioned

Runtime comparison

Before this change: 3m 16s
After this change: 35s

Reviewers: Ming-Yen Chung mingyen066@gmail.com

@github-actions github-actions Bot added triage PRs from the community streams tests Test fixes (including flaky tests) small Small PRs labels Aug 25, 2026
@mingyen066

mingyen066 commented Aug 25, 2026

Copy link
Copy Markdown
Collaborator

Could you combine #23270 and #23251 into this PR? All three make the same changes.

@Russole

Russole commented Aug 25, 2026

Copy link
Copy Markdown
Contributor Author

Closing this PR since the same fix is already covered in #23272,
where the related restart-test changes are grouped together.

@Russole Russole closed this Aug 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

small Small PRs streams tests Test fixes (including flaky tests) triage PRs from the community

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants