[GitHub] [accumulo] dlmarion commented on a diff in pull request #2707: Cached last 64 successful ZKSecurityTool.checkCryptPass password checks

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

dlmarion commented on code in PR #2707:
URL: https://github.com/apache/accumulo/pull/2707#discussion_r882570081


##########
core/src/main/java/org/apache/accumulo/core/conf/Property.java:
##########
@@ -157,6 +157,13 @@ public enum Property {
       "The permission handler class that accumulo will use to determine if a "
           + "user has privilege to perform an action",
       "1.5.0"),
+  INSTANCE_SECURITY_ZK_AUTH_CACHE_ENABLED("instance.security.authenticator.zk.cache.enabled",
+      "true", PropertyType.BOOLEAN,
+      "Enables the temporary caching of successfully authenticated"
+          + " user passwords in org.apache.accumulo.server.security.handler.ZKAuthenticator to"
+          + " mitigate the performance penalties of having to compute the password hash"
+          + " on every API call",
+      "2.1.0"),

Review Comment:
   I reverted the commit that added this property



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

To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org

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