Re: ZooKeeper cwiki - Updated ZooKeeper and SASL auth 1045 work

2017-01-17 Thread Patrick Hunt
Rakesh, send me your cwiki ID and I'll add the proper permissions for you. Patrick On Tue, Jan 17, 2017 at 5:58 PM, Rakesh Radhakrishnan wrote: > Hi PMCs, > > I don't have permission to delete cwiki page. Presently, I have renamed our > old sasl page to "https://cwiki.apache.org/confluence/disp

Re: ZooKeeper cwiki - Updated ZooKeeper and SASL auth 1045 work

2017-01-17 Thread Rakesh Radhakrishnan
Hi PMCs, I don't have permission to delete cwiki page. Presently, I have renamed our old sasl page to "https://cwiki.apache.org/confluence/display/ZOOKEEPER/ Zookeeper+and+SASL-Backup". Could you please delete this page from ZooKeeper project cwiki pages. Thanks! Thanks, Rakesh On Mon, Jan 16, 2

Re: ZooKeeper cwiki - Updated ZooKeeper and SASL auth 1045 work

2017-01-16 Thread Rakesh Radhakrishnan
Hi All, FYI, I'm planning to delete our existing "https://cwiki.apache.org/ confluence/display/ZOOKEEPER/Zookeeper+and+SASL" web page by tomorrow (IST). Then rename https://cwiki.apache.org/confluence/display/ZOOKEEPER/ ZooKeeper+and+SASL+authentication web page to "https://cwiki.apache.org/ conf

Re: ZooKeeper cwiki - Updated ZooKeeper and SASL auth 1045 work

2016-12-20 Thread Rakesh Radhakrishnan
Like I mentioned at the beginning of this mail thread, presently I've maintained this original page as a history. How about deleting this old page now and rename the newly added " https://cwiki.apache.org/confluence/display/ZOOKEEPER/ZooKeeper+and+SASL+authentication"; in place of the old page? I t

Re: ZooKeeper cwiki - Updated ZooKeeper and SASL auth 1045 work

2016-12-19 Thread Patrick Hunt
LGTM. Those changes are very helpful, thanks Rakesh! Patrick On Mon, Dec 19, 2016 at 12:04 PM, Rakesh Radhakrishnan wrote: > Thanks a lot Patrick Hunt for the review comments. Please take another look > at the wiki page when you get a chance. > > I've updated the wiki page addressing these,. >

Re: ZooKeeper cwiki - Updated ZooKeeper and SASL auth 1045 work

2016-12-19 Thread Rakesh Radhakrishnan
Thanks a lot Patrick Hunt for the review comments. Please take another look at the wiki page when you get a chance. I've updated the wiki page addressing these,. 1) ===> DONE. Added JCE encryption part. 2) ===> DONE. Corrected case. 3) ===> DONE. Included version. 4) ===> DONE. Corrected numberin

Re: ZooKeeper cwiki - Updated ZooKeeper and SASL auth 1045 work

2016-12-13 Thread Patrick Hunt
Nice job Rakesh, some comments: 1) the appendix is a great idea, should be useful for many people. One thing I noticed "There is no additional dependencies needed to use SASL with Java since it is part of the the Java Standard Edition." - you might want to mention/link the JCE? The JVM doesn't com

ZooKeeper cwiki - Updated ZooKeeper and SASL auth 1045 work

2016-12-13 Thread Rakesh Radhakrishnan
Hi All, I've incorporated ZK-1045 feature details into the Apache ZooKeeper project cwiki. Since "ZooKeeper and SASL" section is quite large I've splitted ZooKeeper client-server and server-server sections into sub-pages. Please read the following page, https://cwiki.apache.org/confluence/display