[
https://issues.apache.org/jira/browse/KAFKA-4738?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15858581#comment-15858581
]
Guozhang Wang commented on KAFKA-4738:
--------------------------------------
I'd suggest we only remove Generic T in this JIRA, as for Generic C, today it
is using UUID while Integer in the test. UUID is not an ideal unique identifier
for a client so it may be replaced in the future, so keeping Generic C would be
fine for now.
> Remove generic type of class ClientState
> ----------------------------------------
>
> Key: KAFKA-4738
> URL: https://issues.apache.org/jira/browse/KAFKA-4738
> Project: Kafka
> Issue Type: Bug
> Components: streams
> Reporter: Matthias J. Sax
> Assignee: Sharad
> Priority: Minor
> Labels: beginner, newbie
>
> Currently, class
> {{org.apache.kafka.streams.processor.internals.assignment.ClientState<T>}}
> uses a generic type. However, within actual Streams code base the type will
> always be {{TaskId}} (from package {{org.apache.kafka.streams.processor}}).
> Thus, this ticket is about removing the generic type and replace it with
> {{TaskId}}, to simplify the code base.
> There are some tests, that use {{ClientState<Integer>}} (what allows for a
> slightly simplified test setup). Those tests need to be updated to work
> properly using {{TaskId}} instead of {{Integer}}.
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
