cshannon commented on code in PR #5240:
URL: https://github.com/apache/accumulo/pull/5240#discussion_r1909159725
##########
core/src/main/java/org/apache/accumulo/core/metadata/StoredTabletFile.java:
##########
@@ -292,26 +292,26 @@ public TabletFileCq(Path path, Range range) {
}
static class TabletFileCqMetadataGson {
- private String path;
- private byte[] startRow;
- private byte[] endRow;
+ protected String metadataEntry;
+ protected String path;
+ protected byte[] startRow;
+ protected byte[] endRow;
Review Comment:
The new output looks correct to me and what we want, so I think you can go
ahead and remove the separate startRow and endRow as we don't really need to
access it separately as it's part of the metadata path and will be parsed
already when rebuilding StoredTabletFile from the json
--
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]
