milleruntime commented on pull request #2008: URL: https://github.com/apache/accumulo/pull/2008#issuecomment-816771387
> Was the reasoning behind this change discussed elsewhere? The changes look good, just looking for more insight. Nope. I was examining the code while trying to decipher another bug and noticed the two similar constructors. Only having one constructor makes the class easier to follow everywhere it's created (like using find usage in an IDE) and is less code. The other improvements were just bonuses. -- 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]
