This is an automated email from the ASF dual-hosted git repository.
ableegoldman pushed a commit to branch trunk
in repository https://gitbox.apache.org/repos/asf/kafka.git
The following commit(s) were added to refs/heads/trunk by this push:
new 7d1847c4c34 MINOR: Fix KafkaStreams#streamThreadLeaveConsumerGroup
logging (#14526)
7d1847c4c34 is described below
commit 7d1847c4c34abc5dc5657ab8524a5f9d99077e41
Author: Levani Kokhreidze <[email protected]>
AuthorDate: Thu Oct 12 02:14:25 2023 +0300
MINOR: Fix KafkaStreams#streamThreadLeaveConsumerGroup logging (#14526)
Fixes logging for KafkaStreams#streamThreadLeaveConsumerGroup.
In order not to lose the trace of the whole exception, passing Exception e
as a second argument, while message is pre-formatted and passed as string as a
first argument. With this, we won't loose the stack trace of the exception.
Reviewers: Anna Sophie Blee-Goldman <[email protected]>
---
streams/src/main/java/org/apache/kafka/streams/KafkaStreams.java | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/streams/src/main/java/org/apache/kafka/streams/KafkaStreams.java
b/streams/src/main/java/org/apache/kafka/streams/KafkaStreams.java
index 586e0c5f30e..da998d5d983 100644
--- a/streams/src/main/java/org/apache/kafka/streams/KafkaStreams.java
+++ b/streams/src/main/java/org/apache/kafka/streams/KafkaStreams.java
@@ -1548,9 +1548,9 @@ public class KafkaStreams implements AutoCloseable {
removeMembersFromConsumerGroupResult.memberResult(memberToRemove)
.get(remainingTimeMs, TimeUnit.MILLISECONDS);
} catch (final Exception e) {
- log.error("Could not remove static member {} from consumer
group {} due to a: {}",
- groupInstanceId.get(),
-
applicationConfigs.getString(StreamsConfig.APPLICATION_ID_CONFIG), e);
+ final String msg = String.format("Could not remove static
member %s from consumer group %s.",
+ groupInstanceId.get(),
applicationConfigs.getString(StreamsConfig.APPLICATION_ID_CONFIG));
+ log.error(msg, e);
}
}
};