accumulo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From John Vines <vi...@apache.org>
Subject Re: Redundant code in ZKAuthorizor::initializeSecurity ?
Date Thu, 19 Feb 2015 18:23:56 GMT
Yup, they look like they don't belong there.

On Thu, Feb 19, 2015 at 1:06 PM, Srikanth Viswanathan <srikanthv2@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
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message