EdColeman commented on code in PR #17: URL: https://github.com/apache/accumulo-classloaders/pull/17#discussion_r1113146542
########## modules/vfs-class-loader/TESTING.md: ########## @@ -46,12 +46,17 @@ cp ./modules/vfs-class-loader/target/vfs-reloading-classloader-1.0.0-SNAPSHOT.ja Stop Accumulo if it's running and add the following to the accumulo-env.sh: +**Note:** Make sure the commons-vfs2 jar is also on the classpath as well. This dependency was removed in Accumulo 3.0.0 +so if using that version or newer you will need to add it. Also, if you are testing using Uno, do not turn on debug logging +as this will cause errors on start up as the logging to standard out interferes with the configuration parsing. + Review Comment: Is this specific to testing the classloader? Is it the test is reading to logs to config something? As this read now I have the impression that for any testing we cannot set the log level to debug. Maybe as a seperate PR, but if feasible, this seems like something that should be fixed so that debug can be enabled. -- 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]
