mcvsubbu commented on PR #12045: URL: https://github.com/apache/pinot/pull/12045#issuecomment-1831045947
Also, @Jackie-Jiang , it used to be the case originally that the consumer was created in the new thread that is spawned. I think @sajjad-moradi (or, maybe @navina ) moved it to be created in the constructor, which is causing us to handle a new case (I don't remember the reason, but either of them may know). We can also think of moving it back to be created in the consuming thread. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
