bgeng777 commented on a change in pull request #18531:
URL: https://github.com/apache/flink/pull/18531#discussion_r801375581
##########
File path:
flink-yarn/src/main/java/org/apache/flink/yarn/YarnClusterDescriptor.java
##########
@@ -872,14 +872,14 @@ private ApplicationReport startAppMaster(
jobGraph.writeUserArtifactEntriesToConfiguration();
}
-
+ boolean isLocalUsrLibExists = getLocalUsrLibDirectory().isPresent();
Review comment:
Fixed in ef3f0104fc31def407b74b4698d5f7dd0404a333.
##########
File path:
flink-yarn/src/main/java/org/apache/flink/yarn/YarnApplicationFileUploader.java
##########
@@ -438,6 +446,18 @@ private static boolean isPlugin(Path path) {
return false;
}
+ private static boolean isUsrLib(Path path) {
Review comment:
Make sense. Fixed in ef3f0104fc31def407b74b4698d5f7dd0404a333
--
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
