dlmarion commented on a change in pull request #2318:
URL: https://github.com/apache/accumulo/pull/2318#discussion_r742744094
##########
File path: core/src/main/thrift/compaction-coordinator.thrift
##########
@@ -43,9 +43,9 @@ enum TCompactionState {
struct TCompactionStatusUpdate {
1:TCompactionState state
2:string message
- 3:i64 bytesToBeCompacted
- 4:i64 bytesRead
- 5:i64 bytesWritten
+ 3:i64 entriesToBeCompacted
+ 4:i64 entriesRead
+ 5:i64 entriesWritten
}
Review comment:
Yes, I'm pretty sure it's not bytes and it *is* entries. The genesis for
those metrics is
[here](https://github.com/dlmarion/accumulo/blob/expose-external-compaction-progress-for-monitor/server/compactor/src/main/java/org/apache/accumulo/compactor/Compactor.java#L684).
If you dig into that you get to:
FileCompactor.getRunningCompactions() -> new CompactionInfo(FileCompactor)
--
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]
