Skip to content

Commit

Permalink
fix(kafka): ConsumeClaim should not return err when retry recovery fa…
Browse files Browse the repository at this point in the history
…iled
  • Loading branch information
zcong1993 committed Aug 12, 2022
1 parent d30b773 commit 7d96898
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion internal/component/kafka/consumer.go
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ func (consumer *consumer) ConsumeClaim(session sarama.ConsumerGroupSession, clai
}, func() {
consumer.k.logger.Infof("Successfully processed Kafka message after it previously failed: %s/%d/%d [key=%s]", message.Topic, message.Partition, message.Offset, asBase64String(message.Key))
}); err != nil {
return err
consumer.k.logger.Errorf("Error processing Kafka message: %s/%d/%d [key=%s]. Error: %v.", message.Topic, message.Partition, message.Offset, asBase64String(message.Key), err)
}
} else {
err := consumer.doCallback(session, message)
Expand Down

0 comments on commit 7d96898

Please sign in to comment.