milleruntime commented on a change in pull request #2318:
URL: https://github.com/apache/accumulo/pull/2318#discussion_r742336395
##########
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:
I missed why these were changed. I did a rebase on your branch and this
broke some stuff on my branch. I think its OK just wondering. Are these values
not bytes?
--
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]
