cadonna commented on PR #15426: URL: https://github.com/apache/kafka/pull/15426#issuecomment-1966444988
> Thanks for the PR! I left some comments > > Architecturally, this is going a bit against https://issues.apache.org/jira/browse/KAFKA-16290 which proposes propagating the subscription state via events to the application thread, which would resolve the ordering in another way. Do you have thoughts on that? Did we take any decision about this architectural change proposed in https://issues.apache.org/jira/browse/KAFKA-16290? In general I am in favor of the change, but I might miss the reason we opted to use a shared object. @kirktrue @philipnee @lianetm @dajac Does anybody of you know why we chose a shared object instead of sending events between the threads? -- 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
