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

Michael Han commented on ZOOKEEPER-1045:
----------------------------------------

[~rakeshr] Thanks for updating the patch. Latest patch (r6) LGTM and +1.

I noticed that these tests are failing: InvalidSnapshotTest, 
SerializationPerfTest, DeserializationPerfTest. Given that these tests are also 
failing for me locally on branch-3.4 before patch 1045 is applied, I think 
these failures are separate issues. [~rakeshr] Could you please confirm if 
these tests are failing for you as well, before and after apply 1045 to branch 
3.4?

Also FYI I am going to do some high level testing this week as end user, and 
will report back if I find any issues. 



> Quorum Peer mutual authentication
> ---------------------------------
>
>                 Key: ZOOKEEPER-1045
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-1045
>             Project: ZooKeeper
>          Issue Type: New Feature
>          Components: server
>            Reporter: Eugene Koontz
>            Assignee: Rakesh R
>            Priority: Critical
>         Attachments: 0001-ZOOKEEPER-1045-br-3-4.patch, 
> 1045_failing_phunt.tar.gz, ZK-1045-test-case-failure-logs.zip, 
> ZOOKEEPER-1045-00.patch, ZOOKEEPER-1045-Rolling Upgrade Design Proposal.pdf, 
> ZOOKEEPER-1045-br-3-4.patch, ZOOKEEPER-1045-br-3-4.patch, 
> ZOOKEEPER-1045-br-3-4.patch, ZOOKEEPER-1045-br-3-4.patch
>
>
> ZOOKEEPER-938 addresses mutual authentication between clients and servers. 
> This bug, on the other hand, is for authentication among quorum peers. 
> Hopefully much of the work done on SASL integration with Zookeeper for 
> ZOOKEEPER-938 can be used as a foundation for this enhancement.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to