[GitHub] [kafka] showuon commented on a diff in pull request #13050: MINOR: add error reason when controller failed to handle events

[GitBox <gi...@apache.org>]

showuon commented on code in PR #13050:
URL: https://github.com/apache/kafka/pull/13050#discussion_r1059246945


##########
metadata/src/main/java/org/apache/kafka/controller/QuorumController.java:
##########
@@ -432,8 +432,8 @@ private Throwable handleEventException(String name,
                                            OptionalLong startProcessingTimeNs,
                                            Throwable exception) {
         if (!startProcessingTimeNs.isPresent()) {
-            log.error("{}: unable to start processing because of {}.", name,
-                exception.getClass().getSimpleName());
+            log.error("{}: unable to start processing because of {}. Reason: {}", name,

Review Comment:
   Yes, but unsure if that would be too many logs because it'll also log the stack trace. WDYT?



-- 
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.

To unsubscribe, e-mail: jira-unsubscribe@kafka.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org