[
https://issues.apache.org/jira/browse/FLINK-31542?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Flink Jira Bot updated FLINK-31542:
-----------------------------------
Labels: pull-request-available stale-assigned (was: pull-request-available)
I am the [Flink Jira Bot|https://github.com/apache/flink-jira-bot/] and I help
the community manage its development. I see this issue is assigned but has not
received an update in 30 days, so it has been labeled "stale-assigned".
If you are still working on the issue, please remove the label and add a
comment updating the community on your progress. If this issue is waiting on
feedback, please consider this a reminder to the committer/reviewer. Flink is a
very active project, and so we appreciate your patience.
If you are no longer working on the issue, please unassign yourself so someone
else may work on it.
> FatalExceptionClassifier#isFatal returns false if the exception is fatal
> ------------------------------------------------------------------------
>
> Key: FLINK-31542
> URL: https://issues.apache.org/jira/browse/FLINK-31542
> Project: Flink
> Issue Type: Bug
> Components: Connectors / Common
> Affects Versions: 1.16.0
> Reporter: Samuel Siebenmann
> Assignee: DavidLiu
> Priority: Minor
> Labels: pull-request-available, stale-assigned
>
> FatalExceptionClassifier#isFatal returns `false` if the passed throwable is
> fatal and `true` if it is not:
> {code:java}
> public boolean isFatal(Throwable err, Consumer<Exception> throwableConsumer){
>
> if (validator.test(err)) {
> throwableConsumer.accept(throwableMapper.apply(err));
> return false;
> }
> if (chainedClassifier != null) {
> return chainedClassifier.isFatal(err, throwableConsumer);
> } else {
> return true;
> }
> }
> {code}
> ([github|https://github.com/apache/flink/blob/master/flink-connectors/flink-connector-base/src/main/java/org/apache/flink/connector/base/sink/throwable/FatalExceptionClassifier.java#L44])
> However, the semantics of the method would indicate that it should return
> `true` if the passed throwable is fatal and `false` if it is not (i.e. the
> opposite of what is currently the case).
> Additionally, the method name doesn't clearly indicate its side effects.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
