Re: [PR] KAFKA-16655: Deflaking ZKMigrationIntegrationTest.testDualWrite [kafka]

2024-05-03 Thread via GitHub
cmccabe merged PR #15845: URL: https://github.com/apache/kafka/pull/15845 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

Re: [PR] KAFKA-16655: Deflaking ZKMigrationIntegrationTest.testDualWrite [kafka]

2024-05-03 Thread via GitHub
ahuang98 commented on PR #15845: URL: https://github.com/apache/kafka/pull/15845#issuecomment-2093452127 test failures look unrelated -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific

Re: [PR] KAFKA-16655: Deflaking ZKMigrationIntegrationTest.testDualWrite [kafka]

2024-05-02 Thread via GitHub
ahuang98 commented on PR #15845: URL: https://github.com/apache/kafka/pull/15845#issuecomment-2091208654 ^ edited title to kick off another build (last one failed due to unrelated issues) -- This is an automated message from the Apache Git Service. To respond to the message, please log

Re: [PR] KAFKA-16655: Deflaking ZKMigrationIntegrationTest.testDualWrite [kafka]

2024-05-02 Thread via GitHub
ahuang98 commented on code in PR #15845: URL: https://github.com/apache/kafka/pull/15845#discussion_r1588067214 ## core/src/test/scala/integration/kafka/zk/ZkMigrationIntegrationTest.scala: ## @@ -612,7 +613,12 @@ class ZkMigrationIntegrationTest { val readyFuture =

Re: [PR] KAFKA-16655: Deflaking ZKMigrationIntegrationTest.testDualWrite [kafka]

2024-05-02 Thread via GitHub
johnnychhsu commented on PR #15845: URL: https://github.com/apache/kafka/pull/15845#issuecomment-2090696169 thanks for the PR! got a question and left a comment. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

Re: [PR] KAFKA-16655: Deflaking ZKMigrationIntegrationTest.testDualWrite [kafka]

2024-05-02 Thread via GitHub
johnnychhsu commented on code in PR #15845: URL: https://github.com/apache/kafka/pull/15845#discussion_r1587770348 ## core/src/test/scala/integration/kafka/zk/ZkMigrationIntegrationTest.scala: ## @@ -612,7 +613,12 @@ class ZkMigrationIntegrationTest { val readyFuture =

[PR] KAFKA-16655 Deflaking ZKMigrationIntegrationTest.testDualWrite [kafka]

2024-05-01 Thread via GitHub
ahuang98 opened a new pull request, #15845: URL: https://github.com/apache/kafka/pull/15845 This test occasionally fails due to stale broker epoch exceptions, which in turn causes allocate producer ids to fail. Adds retries as stale broker epoch is a retriable issue, and fixes