keith-turner commented on code in PR #5484:
URL: https://github.com/apache/accumulo/pull/5484#discussion_r2047618019
##########
server/manager/src/main/java/org/apache/accumulo/manager/tableOps/bulkVer2/LoadFiles.java:
##########
@@ -142,7 +141,7 @@ public Repo<Manager> call(final long tid, final Manager
manager) {
}
// visible for testing
- public abstract static class Loader {
+ public abstract static class Loader implements AutoCloseable {
Review Comment:
The reason AutoCloseable was added here was to support shutting down the
thread pool using try w/ resources. Did this to avoid leaking threads when
exceptions happen.
--
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]
