EdColeman commented on code in PR #3639:
URL: https://github.com/apache/accumulo/pull/3639#discussion_r1270950955
##########
core/src/main/java/org/apache/accumulo/core/metadata/schema/DataFileValue.java:
##########
@@ -105,7 +105,7 @@ public int hashCode() {
@Override
public String toString() {
- return size + " " + numEntries;
+ return size + " " + numEntries + " " + time;
Review Comment:
I realize that this adds to the existing pattern - but would it be worth it
to use a more standardized toString format that includes the field names?
Otherwise, you need to read the code to understand that '12 1000 1234` is
'size: 12, numEntries 1000, time 1234'
--
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]
