cadonna commented on code in PR #16663:
URL: https://github.com/apache/kafka/pull/16663#discussion_r1689383338
##########
streams/src/main/java/org/apache/kafka/streams/processor/internals/StreamTask.java:
##########
@@ -800,35 +800,39 @@ record = null;
record = null;
throw new TaskCorruptedException(Collections.singleton(id));
}
+ } catch (final FailedProcessingException failedProcessingException) {
+ // Do not keep the failed processing exception in the stack trace
+ handleException(new
Exception(failedProcessingException.getCause()));
} catch (final StreamsException exception) {
record = null;
throw exception;
} catch (final RuntimeException e) {
Review Comment:
Didn't we agree on using `Exception` here instead of `RuntimeException` to
be consistent with other handlers?
--
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
