-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/3631/
-----------------------------------------------------------

(Updated 2012-01-26 00:30:39.276464)


Review request for zookeeper.


Changes
-------

Address Ted and Thomas' reviews.


Summary
-------

ZOOKEEPER-1373: Hardcoded SASL login context name clashes with Hadoop security 
configuration override

Fix is to allow system property to designate the JAAS configuration section 
that the zookeeper client will use.


This addresses bug ZOOKEEPER-1373.
    https://issues.apache.org/jira/browse/ZOOKEEPER-1373


Diffs (updated)
-----

  src/java/test/org/apache/zookeeper/test/SaslAuthFailDesignatedClientTest.java 
PRE-CREATION 
  src/java/test/org/apache/zookeeper/test/SaslAuthMissingClientConfigTest.java 
PRE-CREATION 
  src/java/test/org/apache/zookeeper/test/SaslAuthDesignatedClientTest.java 
PRE-CREATION 
  src/java/main/org/apache/zookeeper/Login.java 6d2a38c 
  src/java/main/org/apache/zookeeper/client/ZooKeeperSaslClient.java 722538e 
  conf/zoo_sample.cfg aafb324 
  src/java/main/org/apache/zookeeper/ClientCnxn.java 6c25e40 

Diff: https://reviews.apache.org/r/3631/diff


Testing
-------

"ant test" java tests pass.


Thanks,

Eugene

Reply via email to