Yup, they look like they don't belong there. On Thu, Feb 19, 2015 at 1:06 PM, Srikanth Viswanathan <srikant...@gmail.com> wrote:
> There appears to be redundant code in the ZKAuthorizor's > 'initializeSecurity' method. It's initializing a bunch of permissions > that are unnecessary in the authorizor. The code seems to have been > replicated from ZKPermHandler. I'd be willing to submit a patch if > someone could confirm that I'm not missing something. > > Code in question: > > > https://github.com/apache/accumulo/blob/master/server/base/src/main/java/org/apache/accumulo/server/security/handler/ZKAuthorizor.java > > (lines 87-94 inclusive) > > Thanks. > > Srikanth >