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
>

Reply via email to