mjsax commented on a change in pull request #9570: URL: https://github.com/apache/kafka/pull/9570#discussion_r518539800
########## File path: streams/src/main/java/org/apache/kafka/streams/processor/internals/RecordCollectorImpl.java ########## @@ -202,6 +204,10 @@ private void recordSendError(final String topic, final Exception exception, fina "indicating the task may be migrated out"; sendException.set(new TaskMigratedException(errorMessage, exception)); } else { + // TODO: KIP-572 handle `TimeoutException extends RetriableException` Review comment: As above: This is an open question. Input is welcome. I would like to tackle it in a follow up PR. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org