iit2009060 commented on PR #21117: URL: https://github.com/apache/kafka/pull/21117#issuecomment-3917730760
> I'm going back and forth on whether we should handle this outside of `handleTimeoutFailure()` or not. On the one hand, the root cause (OOM) is not a timeout, but on the other hand, we might consider hitting an OOM to be a "retriable" error. Thoughts? Yes i think OOM is not a retryable error and we should not be handling in a timeout failure. We should fail fast instead of waiting on the timeout. Should I make a change for this ? -- 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]
