snuyanzin commented on code in PR #22882:
URL: https://github.com/apache/flink/pull/22882#discussion_r1246421258


##########
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 for addressing comment
   I guess I'm not the best person to review here since I 'm not aware of this 
Flink area enough...
   May be another pair of eyes would be good



-- 
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

Reply via email to