cadonna commented on code in PR #14859:
URL: https://github.com/apache/kafka/pull/14859#discussion_r1412110677


##########
clients/src/main/java/org/apache/kafka/clients/consumer/internals/AsyncKafkaConsumer.java:
##########
@@ -663,6 +662,10 @@ public Map<TopicPartition, OffsetAndMetadata> 
committed(final Set<TopicPartition
                     time.timer(timeout));
                 committedOffsets.forEach(this::updateLastSeenEpochIfNewer);
                 return committedOffsets;
+            } catch (TimeoutException e) {
+                throw new TimeoutException("Timeout of " + timeout.toMillis() 
+ "ms expired before the last " +
+                    "committed offset for partitions " + partitions + " could 
be determined. Try tuning default.api.timeout.ms " +

Review Comment:
   While I agree with @kirktrue , let's merge this PR and fix this on the side 
in a follow-up PR.



-- 
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-unsubscr...@kafka.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to