Skip to content

Commit

Permalink
Fix logging for closing partition channels - partition and topic misp…
Browse files Browse the repository at this point in the history
…laced (#977)
  • Loading branch information
sfc-gh-achyzy authored Oct 28, 2024
1 parent 8a7a305 commit ad49fe6
Showing 1 changed file with 4 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -518,8 +518,8 @@ private void closeSequentially(Collection<TopicPartition> partitions) {
LOGGER.info(
"Closing partitionChannel:{}, partition:{}, topic:{}",
topicPartitionChannel == null ? null : topicPartitionChannel.getChannelNameFormatV1(),
topicPartition.topic(),
topicPartition.partition());
topicPartition.partition(),
topicPartition.topic());
partitionsToChannel.remove(partitionChannelKey);
});
}
Expand All @@ -539,8 +539,8 @@ private CompletableFuture<Void> closeTopicPartition(TopicPartition topicPartitio
LOGGER.info(
"Closing partitionChannel:{}, partition:{}, topic:{}",
topicPartitionChannel == null ? null : topicPartitionChannel.getChannelNameFormatV1(),
topicPartition.topic(),
topicPartition.partition());
topicPartition.partition(),
topicPartition.topic());

// It's possible that some partitions can be unassigned before their respective channels are
// even created.
Expand Down

0 comments on commit ad49fe6

Please sign in to comment.