milleruntime commented on a change in pull request #2263:
URL: https://github.com/apache/accumulo/pull/2263#discussion_r705608288



##########
File path: 
server/tserver/src/main/java/org/apache/accumulo/tserver/compactions/CompactionService.java
##########
@@ -363,11 +374,11 @@ private void planCompaction(CompactionKind kind, 
Compactable compactable,
 
       if (!jobs.isEmpty()) {
         log.trace("Submitted compaction plan {} id:{} files:{} plan:{}", 
compactable.getExtent(),
-            myId, files, plan);
+            myId, plan.getCandidates(), plan);

Review comment:
       I reverted the changes to CompactionPlan but here I changed the method 
to use the `Compactable.Files` object as opposed to the Optional. I think this 
cleans it up a bit and should print better info than printing the Optional 
object.




-- 
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]


Reply via email to