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
