paul-rogers commented on a change in pull request #1987: DRILL-7589: Set temporary tests folder for UDF_DIRECTORY_LOCAL, fix allocators closing in BloomFilterTest, fix permissions issue for TestGracefulShutdown tests URL: https://github.com/apache/drill/pull/1987#discussion_r380476183
########## File path: exec/java-exec/src/test/java/org/apache/drill/exec/udf/dynamic/TestDynamicUDFSupport.java ########## @@ -104,8 +104,10 @@ public static void buildAndStoreDefaultJars() throws IOException { @Before public void setupNewDrillbit() throws Exception { udfDir = dirTestWatcher.makeSubDir(Paths.get("udf")); + File udfLocalDir = dirTestWatcher.makeSubDir(Paths.get("udf", "local")); Review comment: The `DirTestWatcher` has internal support for each of Drill's working directories. Might we want to add another directory for UDF files? ---------------------------------------------------------------- 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: us...@infra.apache.org With regards, Apache Git Services