zentol commented on a change in pull request #15553:
URL: https://github.com/apache/flink/pull/15553#discussion_r611469882
##########
File path:
flink-streaming-java/src/main/java/org/apache/flink/streaming/api/operators/collect/CollectResultFetcher.java
##########
@@ -125,7 +127,13 @@ public T next() throws IOException {
try {
response = sendRequest(buffer.getVersion(), requestOffset);
} catch (Exception e) {
- LOG.warn("An exception occurs when fetching query
results", e);
+ if (ExceptionUtils.findThrowable(
+ e,
UnavailableDispatcherOperationException.class)
+ .isPresent()) {
+ LOG.debug("The job execution has not started yet;
cannot fetch results.");
Review comment:
It's difficult to solve, yes. I think we just need to be more conscious
about changing what exceptions we throw; I would hope that in this case,
because it is a fairly specific exception, that we don't have to worry as much.
--
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:
[email protected]
