cshannon commented on issue #18: URL: https://github.com/apache/accumulo-classloaders/issues/18#issuecomment-1444188107
I played around with the prototype a bit and I didn't like writing to a file either from the classloader. Another option that I'm working on is to update the ClusterConfigParser to allow giving a file to output. Right now it always [writes](https://github.com/apache/accumulo/blob/cc44add8bcf9d69057f18d19e6fe9d6b9328575f/core/src/main/java/org/apache/accumulo/core/conf/cluster/ClusterConfigParser.java#L143) to System.out but we could optionally provide an argument for a file that way the configs get written to a file instead so that nothing from Standard.out would interfere. The way it is now if anything (whether it's the VFS classloader or something else) were to write to System.out it could break parsing. -- 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]
