ableegoldman commented on a change in pull request #10000:
URL: https://github.com/apache/kafka/pull/10000#discussion_r570535029
##########
File path:
streams/src/main/java/org/apache/kafka/streams/processor/internals/StreamTask.java
##########
@@ -227,6 +230,27 @@ public void initializeIfNeeded() {
}
}
+ private void initOffsetsIfNeeded(final
java.util.function.Consumer<Set<TopicPartition>> offsetResetter) {
+ final Map<TopicPartition, OffsetAndMetadata> committed =
mainConsumer.committed(resetOffsetsForPartitions);
+ for (final Map.Entry<TopicPartition, OffsetAndMetadata> committedEntry
: committed.entrySet()) {
+ final OffsetAndMetadata offsetAndMetadata =
committedEntry.getValue();
+ if (offsetAndMetadata != null) {
+ mainConsumer.seek(committedEntry.getKey(), offsetAndMetadata);
+ resetOffsetsForPartitions.remove(committedEntry.getKey());
+ }
+ }
+
+ if (!resetOffsetsForPartitions.isEmpty()) {
Review comment:
Can we just pass in a no-op lambda instead? I'd rather avoid special
handling for null input that isn't supposed to be null, just so we can use null
in the tests (which are therefore not realistic tests since it should never be
null, no?)
----------------------------------------------------------------
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]
