[GitHub] [accumulo] ctubbsii commented on issue #1138: Improve HexFormatter scan command

[GitBox <gi...@apache.org>]

ctubbsii commented on issue #1138: Improve HexFormatter scan command
URL: https://github.com/apache/accumulo/issues/1138#issuecomment-487806009
 
 
   Although this might help the hex formatting case... we probably shouldn't design around a single use case. We have to look at the entire configuration space. Lots of little "convenience options" like this could get unwieldy if we don't draw a line somewhere, because everyone's going to want to get theirs baked in.
   
   This is sort of the problem we got into with aggregator configuration in the shell... we had specialized configs for aggregators... but generalized configs for other iterators, including combiners. Although a bit long, the generalized configs were probably better long-run, rather than try to continue to maintain lots of little one-off specialized configs.
   
   That said, a lot of people deal with binary data in Accumulo, and seeing data encoded as base16, hexadecimal, base32, or base64, might be really useful as a general rule to have baked in, so I wouldn't be opposed if we provided those basic facilities, and drew the line on anything else.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services