[GitHub] [accumulo] milleruntime commented on issue #1659: Handle Internal pluggable components

[GitBox <gi...@apache.org>]

milleruntime commented on issue #1659:
URL: https://github.com/apache/accumulo/issues/1659#issuecomment-703807374


   @cradal thanks for the help reviewing internal code but this ticket was more for handling code that currently is NOT part of the API or SPI.  This was to deal with code that should either be part of the public API/SPI or removed from configuration properties, like I did with the MemoryManger.  We had a property for users to write their own MemoryManager but the interface used server components that were not user friendly.  
   
   I think I will have to do the same with the interfaces in org.apache.accumulo.server.security.handler classes (Authenticator, Authorizor, PermissionHandler).  These interfaces are called when Accumulo executes important security procedures (like creating users, initializing a system) but they really only work internally.  It feels like they were added to add the ability for Kerberos to work with the default Zookeeper behavior.  I don't know how we can rework these interfaces to extends this behavior without completely rewriting the interfaces to work in the SPI.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org