[GitHub] storm pull request #2390: [STORM-2790] Add nimbus admins groups

[revans2]

Github user revans2 commented on a diff in the pull request:

    https://github.com/apache/storm/pull/2390#discussion_r147744419
  
    --- Diff: 
storm-client/src/jvm/org/apache/storm/security/auth/authorizer/SimpleACLAuthorizer.java
 ---
    @@ -80,6 +80,7 @@
         }
     
         protected Set<String> _admins;
    +    protected Set<String> _adminsGroups;
    --- End diff --
    
    Same here we should not use '_' as a prefix.


---