You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by GitBox <gi...@apache.org> on 2020/02/19 19:18:09 UTC

[GitHub] [accumulo] alerman opened a new pull request #1522: Make ClientOpts use a password converter that respects the env:PASSWORD

alerman opened a new pull request #1522: Make ClientOpts use a password converter that respects the env:PASSWORD
URL: https://github.com/apache/accumulo/pull/1522
 
 
   ClientOpts doesnt respect env:PASSWORD. We have a requirement to call a class that uses ClientOpts from shell and what to be able to use the environment variable.
   
   This is put up for comment. Open to other suggestions.

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

[GitHub] [accumulo] alerman commented on issue #1522: Make ClientOpts use a password converter that respects the env:PASSWORD

Posted by GitBox <gi...@apache.org>.
alerman commented on issue #1522: Make ClientOpts use a password converter that respects the env:PASSWORD
URL: https://github.com/apache/accumulo/pull/1522#issuecomment-588419403
 
 
   The class that does not support env:<var name> is RemoveEntriesForMissingFiles, which is an accumulo class. We have had a problem with Rfiles periodically going missing, which is an issue. Until that can be fixed we want to be able to run this periodically to get ahead of the compactions failing.

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

[GitHub] [accumulo] alerman edited a comment on issue #1522: Make ClientOpts use a password converter that respects the env:PASSWORD

Posted by GitBox <gi...@apache.org>.
alerman edited a comment on issue #1522: Make ClientOpts use a password converter that respects the env:PASSWORD
URL: https://github.com/apache/accumulo/pull/1522#issuecomment-588419403
 
 
   The class that does not support env:PASS is RemoveEntriesForMissingFiles, which is an accumulo class. We have had a problem with Rfiles periodically going missing, which is an issue. Until that can be fixed we want to be able to run this periodically to get ahead of the compactions failing.

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

[GitHub] [accumulo] alerman closed pull request #1522: Make ClientOpts use a password converter that respects the env:PASSWORD

Posted by GitBox <gi...@apache.org>.
alerman closed pull request #1522: Make ClientOpts use a password converter that respects the env:PASSWORD
URL: https://github.com/apache/accumulo/pull/1522
 
 
   

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

[GitHub] [accumulo] ctubbsii commented on issue #1522: Make ClientOpts use a password converter that respects the env:PASSWORD

Posted by GitBox <gi...@apache.org>.
ctubbsii commented on issue #1522: Make ClientOpts use a password converter that respects the env:PASSWORD
URL: https://github.com/apache/accumulo/pull/1522#issuecomment-588473063
 
 
   Now that I know what class you're using, I feel I must offer a disclaimer. That class has largely remained untouched since the initial public release of Accumulo. It was always intended to be a "use-at-your-own-risk" administrator surgery utility, but since it has largely received on attention across versions which separated the root table, switched to absolute paths, added multiple-volume support, and numerous metadata schema changes, I'd be surprised if it worked reliably and suggest it be used with *extreme* care. It even predates the use of RFiles, still referring to them as "Map Files".
   
   As for its use of the password, as a system utility, it could probably be rewritten to use the system password / system context.
   
   For 2.x, there's no need to add command-line features for passwords, because newer utilities should use either the system user (via the system properties file) or behave as a client (using client properties file, which already has commons-configuration2 features to interface with system environment and property injection).

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

[GitHub] [accumulo] alerman commented on issue #1522: Make ClientOpts use a password converter that respects the env:PASSWORD

Posted by GitBox <gi...@apache.org>.
alerman commented on issue #1522: Make ClientOpts use a password converter that respects the env:PASSWORD
URL: https://github.com/apache/accumulo/pull/1522#issuecomment-588433299
 
 
   I'll make sure this works as expected in 2.0 in which case We can just close this issue and wait for that.

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