bgeng777 commented on a change in pull request #18531: URL: https://github.com/apache/flink/pull/18531#discussion_r801500847
########## File path: flink-yarn/src/main/java/org/apache/flink/yarn/YarnClusterDescriptor.java ########## @@ -1685,6 +1699,35 @@ void addLibFoldersToShipFiles(Collection<File> effectiveShipFiles) { } } + @VisibleForTesting + void addUsrLibFolderToShipFiles( + Collection<File> effectiveShipFiles, Collection<File> systemShipFiles) { + // Add usrlib folder to the ship files if it exists + // Classes in the folder will be loaded by UserClassLoader if CLASSPATH_INCLUDE_USER_JAR is + // DISABLED. + final Optional<File> usrLibDir = getLocalUsrLibDirectory(); + + if (usrLibDir.isPresent()) { + File usrLibDirFile = usrLibDir.get(); + if (usrLibDirFile.isDirectory()) { + checkArgument( Review comment: No problem. In this pr, `ClusterEntrypointUtils.tryFindUserLibDirectory()` is used since 986fff4543fbfd5016f087c53533b395af8c166f. -- 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