milleruntime commented on a change in pull request #2263:
URL: https://github.com/apache/accumulo/pull/2263#discussion_r705617567
##########
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 also noticed printing the `Optional.empty` after we check if its empty
just clutters the log message, so I dropped it:
<pre>
TRACE: Compactable returned no files 4;row_0003250000;row_0003000000 USER
Optional.empty
TRACE: Compactable returned no files 4;row_0003250000;row_0003000000 CHOP
Optional.empty
TRACE: Compactable returned no files 4;row_0003500000;row_0003250000 SYSTEM
Optional.empty
TRACE: Compactable returned no files 4;row_0003500000;row_0003250000
SELECTOR Optional.empty
</pre>
--
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]
