dlmarion commented on code in PR #50:
URL:
https://github.com/apache/accumulo-classloaders/pull/50#discussion_r2709699702
##########
modules/local-caching-classloader/src/main/java/org/apache/accumulo/classloader/lcc/LocalCachingContextClassLoaderFactory.java:
##########
@@ -312,4 +349,17 @@ public static Map<String,List<String>>
getReferencedFiles() {
return referencedContexts;
}
+ private void handleUncaughtExceptionInMonitor() {
+ cleanupLock.writeLock().lock();
+ try {
Review Comment:
@ctubbsii suggested clearing out the context definitions map so that things
would "reset". In an UncaughtExceptionHandler we would not have the context
location, but in the current code we do. I can change this to just clear out
the ContextDefinition for the failure.
--
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]
