sayantanu-dey commented on pull request #11703: URL: https://github.com/apache/kafka/pull/11703#issuecomment-1020797877
> One meta comment regarding consolidating the functions: we have `getPrefix` in multiple classes (and thanks @sayantanu-dey your PRs remove some already), but there are still two duplicated functions in `InternalTopologyBuilder` and `ProcessorContextUtils` as well, could we just keep one and have the other reference it too? Hi, @guozhangwang I have removed the duplicate getPrefix function in `InternalTopologyBuilder` in my latest commit. now there exits only one function `getPrefix` in `ProcessorContextUtils` -- 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