FangYongs commented on code in PR #22882:
URL: https://github.com/apache/flink/pull/22882#discussion_r1246045200
##########
flink-runtime/src/main/java/org/apache/flink/runtime/rest/handler/AbstractHandler.java:
##########
@@ -268,10 +266,7 @@ private CompletableFuture<Void> handleException(
HttpResponseStatus.SERVICE_UNAVAILABLE,
responseHeaders);
} else {
- if (throwable instanceof UnavailableDispatcherOperationException
- || throwable instanceof FileNotFoundException) {
- log.debug("Job is not initialized or is finished: {}",
throwable.getMessage());
- } else {
+ if (log.isDebugEnabled()) {
Review Comment:
Thanks @snuyanzin , good point! I agree with you that we should deal the
exception in `ClientCoordinationsHandler` which is used to return collected
results instead of the generic `AbstractHandler`
--
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: issues-unsubscr...@flink.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
