joshelser commented on pull request #1787: URL: https://github.com/apache/accumulo/pull/1787#issuecomment-729896434
@EdColeman to clarify, the problem with making the change verbatim as it is now (s/256/512/) is understood and not being contested. The question is: if using sha512 instead of sha256 was something which was configurable (opt-in via the user), and there was a contribution for that, would it be accepted? Christopher had stated that he would prefer that not happen, which is how we got into the current discussion. My point was that there is _always_ a "more elegant" solution which can be had. If there is a simple solution which is not harmful, I would err on accepting such a solution (configurable PW Hash algorithm) and letting those with appetite build their elegant solutions on their own time. ---------------------------------------------------------------- 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: [email protected]
