lianetm commented on code in PR #14391: URL: https://github.com/apache/kafka/pull/14391#discussion_r1328961825
########## clients/src/main/java/org/apache/kafka/clients/consumer/internals/events/ApplicationEvent.java: ########## @@ -21,18 +21,27 @@ */ abstract public class ApplicationEvent { public final Type type; + protected final String owner; - protected ApplicationEvent(Type type) { + protected ApplicationEvent(Type type, String owner) { Review Comment: Sure, the point was to avoid the duplicated info in the toString, result of including the class name in the base class and in the child implementations. See my previous [comment](https://github.com/apache/kafka/pull/14391#discussion_r1327561641) related to this down below. -- 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-unsubscr...@kafka.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org