nbali commented on code in PR #26142:
URL: https://github.com/apache/beam/pull/26142#discussion_r1159000162
##########
sdks/java/io/kafka/src/main/java/org/apache/beam/sdk/io/kafka/ReadFromKafkaDoFn.java:
##########
@@ -424,6 +425,29 @@ public ProcessContinuation processElement(
}
}
+ // see https://github.com/apache/beam/issues/25962
+ private ConsumerRecords<byte[], byte[]> poll(
+ Consumer<byte[], byte[]> consumer, TopicPartition topicPartition) {
+ final Stopwatch sw = Stopwatch.createStarted();
+ long previousPosition = -1;
+ while (true) {
+ final ConsumerRecords<byte[], byte[]> rawRecords =
+ consumer.poll(KAFKA_POLL_TIMEOUT.minus(sw.elapsed()));
+ if (!rawRecords.isEmpty()) {
+ // return as we have found some entries
+ return rawRecords;
+ }
+ if (previousPosition == (previousPosition =
consumer.position(topicPartition))) {
Review Comment:
I'm not completely sure we need this. There is a pretty high chance it works
flawlessly even without this.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
