cshannon opened a new issue, #18: URL: https://github.com/apache/accumulo-classloaders/issues/18
Currently debug logging in the AccumuloVFSClassLoader dumps everything into System.out because it can't use log4j as it's not available for the system classloader. So when the [ClusterConfigParser](https://github.com/apache/accumulo/blob/main/core/src/main/java/org/apache/accumulo/core/conf/cluster/ClusterConfigParser.java) is used by the accumulo command to [parse](https://github.com/apache/accumulo/blob/main/assemble/bin/accumulo-cluster#L69) the yaml it breaks as extra debug statements make it into the config file that the scripts are not expecting later when the file is sourced by bash. To fix this we should allow setting a file as a system property that can be used to write log statements to instead of standard out. We can just create a PrintWriter and write to the file directly so that we don't interfere with standard out at all. The debug flag that exists now could just be changed to a file location instead. -- 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]
