[jira] [Commented] (ZOOKEEPER-3418) Improve quorum throughput through eager ACL checks of requests on local servers

["Hudson (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/ZOOKEEPER-3418?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16897881#comment-16897881 ] 

Hudson commented on ZOOKEEPER-3418:
-----------------------------------

FAILURE: Integrated in Jenkins build Zookeeper-trunk-single-thread #475 (See [https://builds.apache.org/job/Zookeeper-trunk-single-thread/475/])
ZOOKEEPER-3418: Improve quorum throughput through eager ACL checks of (eolivelli: rev b2f5548bd5757edbf1887838a487a90523ed2b52)
* (edit) zookeeper-server/src/test/java/org/apache/zookeeper/server/PrepRequestProcessorMetricsTest.java
* (edit) zookeeper-docs/src/main/resources/markdown/zookeeperAdmin.md
* (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/quorum/FollowerRequestProcessor.java
* (add) zookeeper-server/src/test/java/org/apache/zookeeper/server/quorum/EagerACLFilterTest.java
* (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/ZooKeeperServer.java
* (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/DataTree.java
* (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/quorum/LeaderRequestProcessor.java
* (edit) zookeeper-server/src/test/java/org/apache/zookeeper/test/QuorumBase.java
* (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/FinalRequestProcessor.java
* (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/quorum/ObserverRequestProcessor.java
* (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/PrepRequestProcessor.java


> Improve quorum throughput through eager ACL checks of requests on local servers
> -------------------------------------------------------------------------------
>
>                 Key: ZOOKEEPER-3418
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-3418
>             Project: ZooKeeper
>          Issue Type: Improvement
>          Components: server
>    Affects Versions: 3.6.0
>            Reporter: Michael Han
>            Assignee: Michael Han
>            Priority: Major
>              Labels: Twitter, pull-request-available
>             Fix For: 3.6.0
>
>          Time Spent: 3h 40m
>  Remaining Estimate: 0h
>
> Serving write requests that change the state of the system requires quorum operations, and in some cases, the quorum operations can be avoided if the requests are doomed to fail. ACL check failure is such a case. To optimize for this case, we elevate the ACL check logic and perform eager ACL check on local server (where the requests are received), and fail fast, before sending the requests to leader. 
> As with any features, there is a feature flag that can control this feature on, or off (default). This feature is also forward compatible in that for new any new Op code (and some existing Op code we did not explicit check against), they will pass the check and (potentially) fail on leader side, instead of being prematurely filtered out on local server.
> The end result is better throughput and stability of the quorum for certain workloads.



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)