ctubbsii commented on PR #55: URL: https://github.com/apache/accumulo-classloaders/pull/55#issuecomment-3821015507
One thing I noticed while working on this, is that the change to make the DeduplicationCache static, made some test cases a bit confusing... the test directory was blown away between tests, but the test I was writing kept unexpectedly succeeding when I expected an exception, and the reason was because the DeduplicationCache still had entries in it from a previous test. I intend to change it back to non-static once I can implement an alternative to the JMX stuff for easier cleanup (which I am working on next). -- 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]
