davidradl commented on code in PR #26500:
URL: https://github.com/apache/flink/pull/26500#discussion_r2070403876
##########
flink-yarn/src/main/java/org/apache/flink/yarn/YarnClusterDescriptor.java:
##########
@@ -899,14 +889,6 @@ private ApplicationReport startAppMaster(
}
final Set<Path> userJarFiles = new HashSet<>();
- if (jobGraph != null) {
Review Comment:
I see that the variable is not null, I wonder if jobGraphs are intended to
come in in the near future, i would prefer to defer to an owner in this area to
decide whether they want this change
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
