m1a2st commented on PR #17440: URL: https://github.com/apache/kafka/pull/17440#issuecomment-2507785075
Hello @apoorvmittal10, @AndrewJSchofield, @lianetm, @kirktrue The current PR has changed the way metadata errors are propagated, switching from `backgroundEventHandler.add(new ErrorEvent(e))` to using `Optional` for transmission. I noticed that in SharedConsumer, the logic for handling errors on the application thread is slightly different. I think we need to sync up to understand this behavior adjustment better. https://github.com/apache/kafka/blob/b8c8e0c713117ee3f0e8c6c54edd5d2babde5f19/clients/src/main/java/org/apache/kafka/clients/consumer/internals/NetworkClientDelegate.java#L153 -- 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
