AHeise commented on a change in pull request #9959: [FLINK-14235][kafka,tests]
Change source in at-least-once test from finite to infinite
URL: https://github.com/apache/flink/pull/9959#discussion_r337356095
##########
File path:
flink-connectors/flink-connector-kafka-base/src/test/java/org/apache/flink/streaming/connectors/kafka/KafkaProducerTestBase.java
##########
@@ -255,7 +255,7 @@ protected void testOneToOneAtLeastOnce(boolean
regularSink) throws Exception {
Review comment:
FLINK-14235 was another test case.
This test failed because of the race condition of snapshot and EOI.
The other test failed because if this test executed first and failed, proxy
traffic was still blocked (so no data ever arrived at Kafka). Hence, the
successfully written elements are empty ([] in actual).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
