keith-turner commented on code in PR #33: URL: https://github.com/apache/accumulo-classloaders/pull/33#discussion_r2696304229
########## modules/local-caching-classloader/README.md: ########## @@ -170,20 +177,28 @@ constructed at that time. ## Cleanup + Because the cache directory is shared among multiple processes, and one process can't know what the other processes are doing, this class cannot clean up the shared cache directory of unused resources. It is left to the user to remove unused files from the cache. While the context definition JSON files are always safe to delete, it is not recommended to do so for any that are still in use, because they can be useful for troubleshooting. +To aid in this task a JMX MXBean has been created to expose the files that are still referenced Review Comment: > If we can't find one, then we could write one, but I'm not sure we should do that. I agree, we should not write that. If something like that is needed, then it seems like an example is the best we can do. -- 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]
