[GitHub] [accumulo-examples] elinaawise commented on issue #49: Fix #42 Update compactionStrategy example

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

elinaawise commented on issue #49: Fix #42 Update compactionStrategy example
URL: https://github.com/apache/accumulo-examples/pull/49#issuecomment-487052785
 
 
   > More of an observation - passing the secret on the command line is generally discouraged because it will expose the password when it should not be necessary. The Accumulo shell follows the password conventions supported by openssl where the password can be in an environment variable, by a file, or stdin in addition to the command line.
   > 
   > Being this is an example it probably does not make any difference as long as a test cluster is being used, but in practice, this should be avoided and maybe we could point the way by following that in the examples? Or not, it may just complicate things when that's not point of the example.
   
   @EdColeman - I agree and have seen this throughout the examples.  For this example, I have updated the command lines to use <username> instead of "root" and <password> instead of "secret". I will make a note to go back through the rest of the examples and make these updates.
   
   

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