You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jira@kafka.apache.org by "ezhou413 (via GitHub)" <gi...@apache.org> on 2025/10/20 18:31:32 UTC
[PR] Disable `org.apache.kafka.streams.integration.EosIntegrationTest.shouldNotViolateEosIfOneTaskFails` [kafka]
ezhou413 opened a new pull request, #20733:
URL: https://github.com/apache/kafka/pull/20733
Disables `org.apache.kafka.streams.integration.EosIntegrationTest.shouldNotViolateEosIfOneTaskFails` due to flakiness with test runs of the "streams" group protocol.
Ticket to re-enable: https://issues.apache.org/jira/browse/KAFKA-19816
--
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: jira-unsubscribe@kafka.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Disable `org.apache.kafka.streams.integration.EosIntegrationTest.shouldNotViolateEosIfOneTaskFails` [kafka]
Posted by "chia7712 (via GitHub)" <gi...@apache.org>.
chia7712 commented on code in PR #20733:
URL: https://github.com/apache/kafka/pull/20733#discussion_r2445832610
##########
streams/integration-tests/src/test/java/org/apache/kafka/streams/integration/EosIntegrationTest.java:
##########
@@ -395,6 +396,7 @@ public void shouldBeAbleToPerformMultipleTransactions(final String groupProtocol
}
}
+ @Disabled // TODO: KAFKA-19816
Review Comment:
+1 to use `@Flaky("KAFKA-19816")` instead
--
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: jira-unsubscribe@kafka.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] MINOR: Mark`org.apache.kafka.streams.integration.EosIntegrationTest.shouldNotViolateEosIfOneTaskFails` as flaky [kafka]
Posted by "chia7712 (via GitHub)" <gi...@apache.org>.
chia7712 merged PR #20733:
URL: https://github.com/apache/kafka/pull/20733
--
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: jira-unsubscribe@kafka.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Disable `org.apache.kafka.streams.integration.EosIntegrationTest.shouldNotViolateEosIfOneTaskFails` [kafka]
Posted by "lianetm (via GitHub)" <gi...@apache.org>.
lianetm commented on code in PR #20733:
URL: https://github.com/apache/kafka/pull/20733#discussion_r2445847963
##########
streams/integration-tests/src/test/java/org/apache/kafka/streams/integration/EosIntegrationTest.java:
##########
@@ -395,6 +396,7 @@ public void shouldBeAbleToPerformMultipleTransactions(final String groupProtocol
}
}
+ @Disabled // TODO: KAFKA-19816
Review Comment:
Hey @chia7712 ! so good to see you! We haven't been on the same PRs for a while :)
--
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: jira-unsubscribe@kafka.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Disable `org.apache.kafka.streams.integration.EosIntegrationTest.shouldNotViolateEosIfOneTaskFails` [kafka]
Posted by "lianetm (via GitHub)" <gi...@apache.org>.
lianetm commented on code in PR #20733:
URL: https://github.com/apache/kafka/pull/20733#discussion_r2445831085
##########
streams/integration-tests/src/test/java/org/apache/kafka/streams/integration/EosIntegrationTest.java:
##########
@@ -395,6 +396,7 @@ public void shouldBeAbleToPerformMultipleTransactions(final String groupProtocol
}
}
+ @Disabled // TODO: KAFKA-19816
Review Comment:
should we mark it as @Flaky instead? agree it's indeed flaky lately https://develocity.apache.org/scans/tests?search.relativeStartTime=P28D&search.rootProjectNames=kafka&search.tags=trunk&search.timeZoneId=America%2FToronto&tests.container=org.apache.kafka.streams.integration.EosIntegrationTest&tests.sortField=FLAKY&tests.test=shouldNotViolateEosIfOneTaskFails(String%2C%20boolean)%5B3%5D
--
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: jira-unsubscribe@kafka.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] MINOR: Mark`org.apache.kafka.streams.integration.EosIntegrationTest.shouldNotViolateEosIfOneTaskFails` as flaky [kafka]
Posted by "chia7712 (via GitHub)" <gi...@apache.org>.
chia7712 commented on PR #20733:
URL: https://github.com/apache/kafka/pull/20733#issuecomment-3424272068
`shouldNotViolateEosIfOneTaskFails` was successfully moved to the flaky test set. see https://github.com/apache/kafka/actions/runs/18662428370/artifacts/4320782469
--
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: jira-unsubscribe@kafka.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org