[jira] [Commented] (ZOOKEEPER-2103) ZooKeeper Client Configuration
[ https://issues.apache.org/jira/browse/ZOOKEEPER-2103?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15361025#comment-15361025 ] Hadoop QA commented on ZOOKEEPER-2103: -- -1 overall. Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12690650/zookeeper-2103.patch against trunk revision 1750739. +1 @author. The patch does not contain any @author tags. +1 tests included. The patch appears to include 24 new or modified tests. -1 patch. The patch command could not apply the patch. Console output: https://builds.apache.org/job/PreCommit-ZOOKEEPER-Build/3260//console This message is automatically generated. > ZooKeeper Client Configuration > -- > > Key: ZOOKEEPER-2103 > URL: https://issues.apache.org/jira/browse/ZOOKEEPER-2103 > Project: ZooKeeper > Issue Type: New Feature > Components: java client >Affects Versions: 3.6.0 > Environment: All >Reporter: Chris Larsen >Assignee: Chris Larsen >Priority: Minor > Labels: features, patch > Fix For: 3.6.0 > > Attachments: zookeeper-2103.patch > > > I ran into an issue when connecting to two ZooKeeper clusters from the same > JVM application. One of the clusters required SASL authentication while the > other one did not. Unfortunately the client uses System properties to > configure authentication and the client was attempting to authenticate on the > non-auth cluster, preventing a connection. > To solve it, I implemented a base config class with helper methods for > parsing config settings as well as a client specific subclass that parsed the > system system values but allowed for overriding via programatic values or via > a file. There are also new Zookeeper constructors to use this config object. > I implemented it so that it's completely backwards compatible so it shouldn't > break existing installs (and it hasn't yet with my testing). > If folks like this, we could use the same config base for server configs and > migrate away from system properties to per object configs. It would also be > helpful to centralize more of the "zookeeper.*" strings. > Let me know what ya'll think and thanks! -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (ZOOKEEPER-2103) ZooKeeper Client Configuration
[ https://issues.apache.org/jira/browse/ZOOKEEPER-2103?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15361019#comment-15361019 ] Arshad Mohammad commented on ZOOKEEPER-2103: ZOOKEEPER-2139 has already implemented the functionality expected in this jira [~manolamancha], can you please verify and close this jira > ZooKeeper Client Configuration > -- > > Key: ZOOKEEPER-2103 > URL: https://issues.apache.org/jira/browse/ZOOKEEPER-2103 > Project: ZooKeeper > Issue Type: New Feature > Components: java client >Affects Versions: 3.6.0 > Environment: All >Reporter: Chris Larsen >Assignee: Chris Larsen >Priority: Minor > Labels: features, patch > Fix For: 3.6.0 > > Attachments: zookeeper-2103.patch > > > I ran into an issue when connecting to two ZooKeeper clusters from the same > JVM application. One of the clusters required SASL authentication while the > other one did not. Unfortunately the client uses System properties to > configure authentication and the client was attempting to authenticate on the > non-auth cluster, preventing a connection. > To solve it, I implemented a base config class with helper methods for > parsing config settings as well as a client specific subclass that parsed the > system system values but allowed for overriding via programatic values or via > a file. There are also new Zookeeper constructors to use this config object. > I implemented it so that it's completely backwards compatible so it shouldn't > break existing installs (and it hasn't yet with my testing). > If folks like this, we could use the same config base for server configs and > migrate away from system properties to per object configs. It would also be > helpful to centralize more of the "zookeeper.*" strings. > Let me know what ya'll think and thanks! -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (ZOOKEEPER-2103) ZooKeeper Client Configuration
[ https://issues.apache.org/jira/browse/ZOOKEEPER-2103?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15177258#comment-15177258 ] Hadoop QA commented on ZOOKEEPER-2103: -- -1 overall. Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12690650/zookeeper-2103.patch against trunk revision 1733348. +1 @author. The patch does not contain any @author tags. +1 tests included. The patch appears to include 24 new or modified tests. -1 patch. The patch command could not apply the patch. Console output: https://builds.apache.org/job/PreCommit-ZOOKEEPER-Build/3068//console This message is automatically generated. > ZooKeeper Client Configuration > -- > > Key: ZOOKEEPER-2103 > URL: https://issues.apache.org/jira/browse/ZOOKEEPER-2103 > Project: ZooKeeper > Issue Type: New Feature > Components: java client >Affects Versions: 3.6.0 > Environment: All >Reporter: Chris Larsen >Assignee: Chris Larsen >Priority: Minor > Labels: features, patch > Fix For: 3.6.0 > > Attachments: zookeeper-2103.patch > > > I ran into an issue when connecting to two ZooKeeper clusters from the same > JVM application. One of the clusters required SASL authentication while the > other one did not. Unfortunately the client uses System properties to > configure authentication and the client was attempting to authenticate on the > non-auth cluster, preventing a connection. > To solve it, I implemented a base config class with helper methods for > parsing config settings as well as a client specific subclass that parsed the > system system values but allowed for overriding via programatic values or via > a file. There are also new Zookeeper constructors to use this config object. > I implemented it so that it's completely backwards compatible so it shouldn't > break existing installs (and it hasn't yet with my testing). > If folks like this, we could use the same config base for server configs and > migrate away from system properties to per object configs. It would also be > helpful to centralize more of the "zookeeper.*" strings. > Let me know what ya'll think and thanks! -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (ZOOKEEPER-2103) ZooKeeper Client Configuration
[ https://issues.apache.org/jira/browse/ZOOKEEPER-2103?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14268540#comment-14268540 ] Hadoop QA commented on ZOOKEEPER-2103: -- +1 overall. Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12690650/zookeeper-2103.patch against trunk revision 1646992. +1 @author. The patch does not contain any @author tags. +1 tests included. The patch appears to include 24 new or modified tests. +1 javadoc. The javadoc tool did not generate any warning messages. +1 javac. The applied patch does not increase the total number of javac compiler warnings. +1 findbugs. The patch does not introduce any new Findbugs (version 2.0.3) warnings. +1 release audit. The applied patch does not increase the total number of release audit warnings. +1 core tests. The patch passed core unit tests. +1 contrib tests. The patch passed contrib unit tests. Test results: https://builds.apache.org/job/PreCommit-ZOOKEEPER-Build/2479//testReport/ Findbugs warnings: https://builds.apache.org/job/PreCommit-ZOOKEEPER-Build/2479//artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html Console output: https://builds.apache.org/job/PreCommit-ZOOKEEPER-Build/2479//console This message is automatically generated. > ZooKeeper Client Configuration > -- > > Key: ZOOKEEPER-2103 > URL: https://issues.apache.org/jira/browse/ZOOKEEPER-2103 > Project: ZooKeeper > Issue Type: New Feature > Components: java client >Affects Versions: 3.6.0 > Environment: All >Reporter: Chris Larsen >Priority: Minor > Labels: features, patch > Fix For: 3.6.0 > > Attachments: zookeeper-2103.patch > > > I ran into an issue when connecting to two ZooKeeper clusters from the same > JVM application. One of the clusters required SASL authentication while the > other one did not. Unfortunately the client uses System properties to > configure authentication and the client was attempting to authenticate on the > non-auth cluster, preventing a connection. > To solve it, I implemented a base config class with helper methods for > parsing config settings as well as a client specific subclass that parsed the > system system values but allowed for overriding via programatic values or via > a file. There are also new Zookeeper constructors to use this config object. > I implemented it so that it's completely backwards compatible so it shouldn't > break existing installs (and it hasn't yet with my testing). > If folks like this, we could use the same config base for server configs and > migrate away from system properties to per object configs. It would also be > helpful to centralize more of the "zookeeper.*" strings. > Let me know what ya'll think and thanks! -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (ZOOKEEPER-2103) ZooKeeper Client Configuration
[ https://issues.apache.org/jira/browse/ZOOKEEPER-2103?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14268445#comment-14268445 ] Hadoop QA commented on ZOOKEEPER-2103: -- -1 overall. Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12690637/zookeeper-2103.patch against trunk revision 1646992. +1 @author. The patch does not contain any @author tags. +1 tests included. The patch appears to include 24 new or modified tests. +1 javadoc. The javadoc tool did not generate any warning messages. +1 javac. The applied patch does not increase the total number of javac compiler warnings. -1 findbugs. The patch appears to introduce 3 new Findbugs (version 2.0.3) warnings. +1 release audit. The applied patch does not increase the total number of release audit warnings. -1 core tests. The patch failed core unit tests. +1 contrib tests. The patch passed contrib unit tests. Test results: https://builds.apache.org/job/PreCommit-ZOOKEEPER-Build/2478//testReport/ Findbugs warnings: https://builds.apache.org/job/PreCommit-ZOOKEEPER-Build/2478//artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html Console output: https://builds.apache.org/job/PreCommit-ZOOKEEPER-Build/2478//console This message is automatically generated. > ZooKeeper Client Configuration > -- > > Key: ZOOKEEPER-2103 > URL: https://issues.apache.org/jira/browse/ZOOKEEPER-2103 > Project: ZooKeeper > Issue Type: New Feature > Components: java client >Affects Versions: 3.6.0 > Environment: All >Reporter: Chris Larsen >Priority: Minor > Labels: features, patch > Fix For: 3.6.0 > > Attachments: zookeeper-2103.patch > > > I ran into an issue when connecting to two ZooKeeper clusters from the same > JVM application. One of the clusters required SASL authentication while the > other one did not. Unfortunately the client uses System properties to > configure authentication and the client was attempting to authenticate on the > non-auth cluster, preventing a connection. > To solve it, I implemented a base config class with helper methods for > parsing config settings as well as a client specific subclass that parsed the > system system values but allowed for overriding via programatic values or via > a file. There are also new Zookeeper constructors to use this config object. > I implemented it so that it's completely backwards compatible so it shouldn't > break existing installs (and it hasn't yet with my testing). > If folks like this, we could use the same config base for server configs and > migrate away from system properties to per object configs. It would also be > helpful to centralize more of the "zookeeper.*" strings. > Let me know what ya'll think and thanks! -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (ZOOKEEPER-2103) ZooKeeper Client Configuration
[ https://issues.apache.org/jira/browse/ZOOKEEPER-2103?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14268400#comment-14268400 ] Hadoop QA commented on ZOOKEEPER-2103: -- -1 overall. Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12690632/zookeeper-2103.patch against trunk revision 1646992. +1 @author. The patch does not contain any @author tags. +1 tests included. The patch appears to include 24 new or modified tests. +1 javadoc. The javadoc tool did not generate any warning messages. +1 javac. The applied patch does not increase the total number of javac compiler warnings. -1 findbugs. The patch appears to introduce 3 new Findbugs (version 2.0.3) warnings. +1 release audit. The applied patch does not increase the total number of release audit warnings. -1 core tests. The patch failed core unit tests. +1 contrib tests. The patch passed contrib unit tests. Test results: https://builds.apache.org/job/PreCommit-ZOOKEEPER-Build/2477//testReport/ Findbugs warnings: https://builds.apache.org/job/PreCommit-ZOOKEEPER-Build/2477//artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html Console output: https://builds.apache.org/job/PreCommit-ZOOKEEPER-Build/2477//console This message is automatically generated. > ZooKeeper Client Configuration > -- > > Key: ZOOKEEPER-2103 > URL: https://issues.apache.org/jira/browse/ZOOKEEPER-2103 > Project: ZooKeeper > Issue Type: New Feature > Components: java client >Affects Versions: 3.6.0 > Environment: All >Reporter: Chris Larsen >Priority: Minor > Labels: features, patch > Fix For: 3.6.0 > > Attachments: zookeeper-2103.patch > > > I ran into an issue when connecting to two ZooKeeper clusters from the same > JVM application. One of the clusters required SASL authentication while the > other one did not. Unfortunately the client uses System properties to > configure authentication and the client was attempting to authenticate on the > non-auth cluster, preventing a connection. > To solve it, I implemented a base config class with helper methods for > parsing config settings as well as a client specific subclass that parsed the > system system values but allowed for overriding via programatic values or via > a file. There are also new Zookeeper constructors to use this config object. > I implemented it so that it's completely backwards compatible so it shouldn't > break existing installs (and it hasn't yet with my testing). > If folks like this, we could use the same config base for server configs and > migrate away from system properties to per object configs. It would also be > helpful to centralize more of the "zookeeper.*" strings. > Let me know what ya'll think and thanks! -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (ZOOKEEPER-2103) ZooKeeper Client Configuration
[ https://issues.apache.org/jira/browse/ZOOKEEPER-2103?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14268302#comment-14268302 ] Hadoop QA commented on ZOOKEEPER-2103: -- -1 overall. Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12690617/zookeeper-2103.diff against trunk revision 1646992. +1 @author. The patch does not contain any @author tags. +1 tests included. The patch appears to include 24 new or modified tests. +1 javadoc. The javadoc tool did not generate any warning messages. +1 javac. The applied patch does not increase the total number of javac compiler warnings. -1 findbugs. The patch appears to introduce 3 new Findbugs (version 2.0.3) warnings. +1 release audit. The applied patch does not increase the total number of release audit warnings. +1 core tests. The patch passed core unit tests. +1 contrib tests. The patch passed contrib unit tests. Test results: https://builds.apache.org/job/PreCommit-ZOOKEEPER-Build/2476//testReport/ Findbugs warnings: https://builds.apache.org/job/PreCommit-ZOOKEEPER-Build/2476//artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html Console output: https://builds.apache.org/job/PreCommit-ZOOKEEPER-Build/2476//console This message is automatically generated. > ZooKeeper Client Configuration > -- > > Key: ZOOKEEPER-2103 > URL: https://issues.apache.org/jira/browse/ZOOKEEPER-2103 > Project: ZooKeeper > Issue Type: New Feature > Components: java client >Affects Versions: 3.6.0 > Environment: All >Reporter: Chris Larsen >Priority: Minor > Labels: features, patch > Fix For: 3.6.0 > > Attachments: zookeeper-2103.patch > > > I ran into an issue when connecting to two ZooKeeper clusters from the same > JVM application. One of the clusters required SASL authentication while the > other one did not. Unfortunately the client uses System properties to > configure authentication and the client was attempting to authenticate on the > non-auth cluster, preventing a connection. > To solve it, I implemented a base config class with helper methods for > parsing config settings as well as a client specific subclass that parsed the > system system values but allowed for overriding via programatic values or via > a file. There are also new Zookeeper constructors to use this config object. > I implemented it so that it's completely backwards compatible so it shouldn't > break existing installs (and it hasn't yet with my testing). > If folks like this, we could use the same config base for server configs and > migrate away from system properties to per object configs. It would also be > helpful to centralize more of the "zookeeper.*" strings. > Let me know what ya'll think and thanks! -- This message was sent by Atlassian JIRA (v6.3.4#6332)