kirktrue commented on PR #15426: URL: https://github.com/apache/kafka/pull/15426#issuecomment-1968017311
> Did we take any decision about this architectural change proposed in [issues.apache.org/jira/browse/KAFKA-16290](https://issues.apache.org/jira/browse/KAFKA-16290)? Not that I'm aware of, no. I grabbed the ticket so that it would show up on my list of things to worry about 😟 > @kirktrue @philipnee @lianetm @dajac Does anybody of you know why we chose a shared object instead of sending events between the threads? IIRC, there were two reasons we felt comfortable using the `SubscriptionState` object directly: 1. It is heavily synchronized 2. We didn't want to add overhead for simple operations like looking up the subscribed topics, etc. -- 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
