root
/
kafka
mirror of https://github.com/apache/kafka.git
1
0
Fork 0
Code Issues Actions 2 Packages Projects Releases Wiki Activity

MINOR: fix incorrect offset reset logging (#20558) 

We need to only pass in the reset strategy, as the `logMessage`
parameter was removed.

Reviewers: Chia-Ping Tsai <chia7712@gmail.com>, Lucas Brutschy
 <lbrutschy@confluent.io>
This commit is contained in:
Matthias J. Sax 2025-09-22 09:54:50 -07:00 committed by GitHub
parent f16d1f3c9d
commit 71efb89290
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 6 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
streams/src/main/java/org/apache/kafka/streams/processor/internals/StreamThread.java
View File

@ -1532,6 +1532,7 @@ public class StreamThread extends Thread implements ProcessingThread {
try { try {
records = mainConsumer.poll(pollTime); records = mainConsumer.poll(pollTime);
} catch (final InvalidOffsetException e) { } catch (final InvalidOffsetException e) {
log.info("Found no valid offset for {} partitions, resetting.", e.partitions().size());
resetOffsets(e.partitions(), e); resetOffsets(e.partitions(), e);
} }
@ -1647,14 +1648,14 @@ public class StreamThread extends Thread implements ProcessingThread {
addToResetList( addToResetList(
partition, partition,
seekToBeginning, seekToBeginning,
"Setting topic '{}' to consume from earliest offset", "Setting topic '{}' to consume from 'earliest' offset",
loggedTopics loggedTopics
); );
} else if (resetPolicy == AutoOffsetResetStrategy.LATEST) { } else if (resetPolicy == AutoOffsetResetStrategy.LATEST) {
addToResetList( addToResetList(
partition, partition,
seekToEnd, seekToEnd,
"Setting topic '{}' to consume from latest offset", "Setting topic '{}' to consume from 'latest' offset",
loggedTopics loggedTopics
); );
} else if (resetPolicy.type() == AutoOffsetResetStrategy.StrategyType.BY_DURATION) { } else if (resetPolicy.type() == AutoOffsetResetStrategy.StrategyType.BY_DURATION) {
@ -1662,7 +1663,7 @@ public class StreamThread extends Thread implements ProcessingThread {
partition, partition,
seekByDuration, seekByDuration,
resetPolicy.duration().get(), resetPolicy.duration().get(),
"Setting topic '{}' to consume from by_duration:{}", "Setting topic '{}' to consume from 'by_duration:{}'",
resetPolicy.duration().get().toString(), resetPolicy.duration().get().toString(),
loggedTopics loggedTopics
); );
@ -1778,12 +1779,12 @@ public class StreamThread extends Thread implements ProcessingThread {
private void addToResetList( private void addToResetList(
final TopicPartition partition, final TopicPartition partition,
final Set<TopicPartition> partitions, final Set<TopicPartition> partitions,
final String resetPolicy, final String logMessage,
final Set<String> loggedTopics final Set<String> loggedTopics
) { ) {
final String topic = partition.topic(); final String topic = partition.topic();
if (loggedTopics.add(topic)) { if (loggedTopics.add(topic)) {
log.info("Setting topic '{}' to consume from {} offset", topic, resetPolicy); log.info(logMessage, topic);
} }
partitions.add(partition); partitions.add(partition);
} }