ableegoldman commented on a change in pull request #10072:
URL: https://github.com/apache/kafka/pull/10072#discussion_r571317046
##########
File path: streams/src/main/java/org/apache/kafka/streams/processor/TaskId.java
##########
@@ -32,6 +33,7 @@
public final int topicGroupId;
/** The ID of the partition. */
public final int partition;
+ public Task task;
Review comment:
I have to say, it makes me a little uncomfortable to stick the actual
`Task` object inside the basic `TaskId` container class. Especially if 99% of
the time it will be null, given that we use `TaskId` all over the place and
only call `setTask` a handful of time. It will only get increasingly difficult
about whether it's safe to assume a given `TaskId` object has an actual
non-null `Task` or not, and therefore is safe to use 😬
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
