cshannon commented on code in PR #5240:
URL: https://github.com/apache/accumulo/pull/5240#discussion_r1909204936


##########
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:
   Could we write a tool or something to make scanning and reading the entry 
easier? I think there was talk of bringing back formatters for the console. We 
will need something else to be able to read the base64 range that is encoded 
anyways so we could just combine it all as part of that and revert #5237 
   
   There is already other issues related to being able to read the metadata so 
this could be combined: #3767 and #3772



-- 
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]

Reply via email to