You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@accumulo.apache.org by GitBox <gi...@apache.org> on 2020/10/06 15:07:19 UTC

[GitHub] [accumulo] milleruntime commented on pull request #1721: Change security classname properties to boolean

milleruntime commented on pull request #1721:
URL: https://github.com/apache/accumulo/pull/1721#issuecomment-704335397


   > Until that comprehensive work is done, I think these should remain non-public API, and left as an advanced / marginally supported feature through the "internal" APIs. If it's "easy" to make the existing interfaces themselves public API / SPI, without exposing additional internal types, I'd be okay with that change also.
   
   I made attempts at trying to modify them to fit our API or SPI.  The SPI may be more feasible but the main problem is their implementation relies heavily on access to internal ZooKeeper objects.  Most of the work the classes are doing is just interacting with zookeeper.  The Kerberos impl still uses the ZK impl but with added checks.  So I don't think there is an "easy" approach.


----------------------------------------------------------------
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