keith-turner opened a new pull request, #6097: URL: https://github.com/apache/accumulo/pull/6097
When the property `general.vfs.classpaths` was not set VFS was still unnecessarily initialized. Modified to only initialized VFS when the property is actually set. Tested locally by setting `general.vfs.classpaths=`, `general.dynamic.classpaths=`, `general.context.class.loader.factory=TestCL`. Where TestCL is a classloader that does nothing. When all of these are set did not see VFS create anything in local filesystem. Had to set some props to empty string because mini accumulo would set them otherwise. Also added some debug logging if VFS was initialized and did not see that. Removed that logging from these changes. -- 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]
