ctubbsii commented on a change in pull request #870: Remove unused code in 
master and server
URL: https://github.com/apache/accumulo/pull/870#discussion_r245433910
 
 

 ##########
 File path: 
server/base/src/main/java/org/apache/accumulo/server/security/handler/PermissionHandler.java
 ##########
 @@ -138,11 +138,6 @@ void cleanNamespacePermissions(Namespace.ID namespace)
    */
   void initUser(String user) throws AccumuloSecurityException;
 
-  /**
-   * Initializes a new user
-   */
-  void initTable(String table);
 
 Review comment:
   This was originally supposed to be called when a new table was created... to 
initialize the permissions set in whatever underlying implemented storage. In 
our implementations, I think it wasn't needed, so it should have been a NOOP, 
but it should still have been called (I think).

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to