Re: [PR] Human-readable metadata scan [accumulo]

["ArbaazKhan1 (via GitHub)" <gi...@apache.org>]

ArbaazKhan1 commented on code in PR #6519:
URL: https://github.com/apache/accumulo/pull/6519#discussion_r3981957192


##########
shell/src/main/java/org/apache/accumulo/shell/commands/ScanCommand.java:
##########
@@ -352,6 +400,8 @@ public Options getOptions() {
     executionHintsOpt = new Option(null, "execution-hints", true, "Execution hints map");
     scanServerOpt =
         new Option("cl", "consistency-level", true, "set consistency level (experimental)");
+    decodeMetadataOpt = new Option("dm", "decode-metadata", false,

Review Comment:
   My goal was to avoid changing the normal metadata scan output in a way that might make users think that the decoded form is the actual stored metadata representation. I figured adding a new option, allows for the default behavior to remain unchanged, and users only get the decoded/human-readable form when they explicitly ask for it



-- 
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: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org