[jira] Commented: (ZOOKEEPER-78) added a high level protocol/feature - for easy Leader Election or exclusive Write Lock creation
[ https://issues.apache.org/jira/browse/ZOOKEEPER-78?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12701685#action_12701685 ] Flavio Paiva Junqueira commented on ZOOKEEPER-78: - +1, it looks good to me. > added a high level protocol/feature - for easy Leader Election or exclusive > Write Lock creation > --- > > Key: ZOOKEEPER-78 > URL: https://issues.apache.org/jira/browse/ZOOKEEPER-78 > Project: Zookeeper > Issue Type: New Feature > Components: java client >Affects Versions: 3.0.0 >Reporter: james strachan >Assignee: Mahadev konar > Fix For: 3.2.0 > > Attachments: patch_with_including_Benjamin's_fix.patch, > using_zookeeper_facade.patch, ZOOKEEPER-78.patch, ZOOKEEPER-78.patch, > ZOOKEEPER-78.patch, ZOOKEEPER-78.patch, ZOOKEEPER-78.patch, > ZOOKEEPER-78.patch, ZOOKEEPER-78.patch, ZOOKEEPER-78.patch, ZOOKEEPER-78.patch > > > Here's a patch which adds a little WriteLock helper class for performing > leader elections or creating exclusive locks in some directory znode. Note > its an early cut; am sure we can improve it over time. The aim is to avoid > folks having to use the low level ZK stuff but provide a simpler high level > abstraction. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.
[jira] Commented: (ZOOKEEPER-36) REST access to ZooKeeper
[ https://issues.apache.org/jira/browse/ZOOKEEPER-36?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12701616#action_12701616 ] Patrick Hunt commented on ZOOKEEPER-36: --- Thanks for further input Chris, really appreciate you giving insight. I'm not an expert at REST so please feel free. WRT webdav - jersey has support for webdav being developed. I think it makes sense to have webdav client support (it's in windows for example) at some point. I'm focusing on rest initially in order to cover as many dev environments (scripting in particular) and developer experience as possible (devs more expr with rest than webdav typically) Also, I'm not sure how well zk fits into webdav given that we store data on znodes that can act as either directories or files. Perhaps we just need a "phantom" file that represents the data associated with a non-leaf znode. I did think about MKCOL initially, actually I was thinking of using it even in this JIRA, but in the end decided not to in order to stick within the confines of std REST. > REST access to ZooKeeper > > > Key: ZOOKEEPER-36 > URL: https://issues.apache.org/jira/browse/ZOOKEEPER-36 > Project: Zookeeper > Issue Type: New Feature > Components: contrib >Reporter: Patrick Hunt >Assignee: Patrick Hunt > Fix For: 3.2.0 > > Attachments: rest_2.tar.gz, SPEC.txt > > > Moved from SourceForge to Apache. > http://sourceforge.net/tracker/index.php?func=detail&aid=1961763&group_id=209147&atid=1008547 -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.
[jira] Updated: (ZOOKEEPER-371) to implement jdiff
[ https://issues.apache.org/jira/browse/ZOOKEEPER-371?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Patrick Hunt updated ZOOKEEPER-371: --- Resolution: Fixed Hadoop Flags: [Reviewed] Status: Resolved (was: Patch Available) Committed revision 767596 Thanks Giri! > to implement jdiff > --- > > Key: ZOOKEEPER-371 > URL: https://issues.apache.org/jira/browse/ZOOKEEPER-371 > Project: Zookeeper > Issue Type: Improvement > Components: build >Affects Versions: 3.2.0 > Environment: to implement jdiff >Reporter: Giridharan Kesavan >Assignee: Giridharan Kesavan > Fix For: 3.2.0 > > Attachments: zookeeper-371.patch, zookeeper-371.patch > > -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.
[jira] Commented: (ZOOKEEPER-371) to implement jdiff
[ https://issues.apache.org/jira/browse/ZOOKEEPER-371?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12701610#action_12701610 ] Patrick Hunt commented on ZOOKEEPER-371: +1, looks great. I generated the release package and it looks good. Thanks Giri! > to implement jdiff > --- > > Key: ZOOKEEPER-371 > URL: https://issues.apache.org/jira/browse/ZOOKEEPER-371 > Project: Zookeeper > Issue Type: Improvement > Components: build >Affects Versions: 3.2.0 > Environment: to implement jdiff >Reporter: Giridharan Kesavan >Assignee: Giridharan Kesavan > Fix For: 3.2.0 > > Attachments: zookeeper-371.patch, zookeeper-371.patch > > -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.
[jira] Commented: (ZOOKEEPER-377) running ant cppunit tests, a failure still results in BUILD SUCCESSFUL
[ https://issues.apache.org/jira/browse/ZOOKEEPER-377?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12701558#action_12701558 ] Hudson commented on ZOOKEEPER-377: -- Integrated in ZooKeeper-trunk #287 (See [http://hudson.zones.apache.org/hudson/job/ZooKeeper-trunk/287/]) > running ant cppunit tests, a failure still results in BUILD SUCCESSFUL > -- > > Key: ZOOKEEPER-377 > URL: https://issues.apache.org/jira/browse/ZOOKEEPER-377 > Project: Zookeeper > Issue Type: Bug >Affects Versions: 3.2.0 >Reporter: Patrick Hunt >Assignee: Giridharan Kesavan >Priority: Critical > Fix For: 3.2.0 > > Attachments: zookeeper-377.patch > > > Giri, can you take a look at this? > I ran "ant test" > looks like this test failed: > [exec] Zookeeper_operations::testOperationsAndDisconnectConcurrently1 : > assertion > > then later... > [exec] > [exec] > /home/phunt/dev/workspace/svnzk_apache/src/c/tests/TestOperations.cc:551: > Assertion: equality assertion failed [Expected: -4, Actual : 0, > ZCONNECTIONLOSS != rc] > [exec] Failures !!! > [exec] Run: 38 Failure total: 1 Failures: 1 Errors: 0 > [exec] make: *** [run-check] Error 1 > [exec] Result: 2 > test-core: > test-contrib: > BUILD SUCCESSFUL > Total time: 15 minutes 39 seconds -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.
Hudson build is back to normal: ZooKeeper-trunk #287
See http://hudson.zones.apache.org/hudson/job/ZooKeeper-trunk/287/
[jira] Commented: (ZOOKEEPER-29) Flexible quorums
[ https://issues.apache.org/jira/browse/ZOOKEEPER-29?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12701553#action_12701553 ] Hadoop QA commented on ZOOKEEPER-29: +1 overall. Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12406121/ZOOKEEPER-29.patch against trunk revision 766897. +1 @author. The patch does not contain any @author tags. +1 tests included. The patch appears to include 3 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 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: http://hudson.zones.apache.org/hudson/job/Zookeeper-Patch-vesta.apache.org/47/testReport/ Findbugs warnings: http://hudson.zones.apache.org/hudson/job/Zookeeper-Patch-vesta.apache.org/47/artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html Console output: http://hudson.zones.apache.org/hudson/job/Zookeeper-Patch-vesta.apache.org/47/console This message is automatically generated. > Flexible quorums > > > Key: ZOOKEEPER-29 > URL: https://issues.apache.org/jira/browse/ZOOKEEPER-29 > Project: Zookeeper > Issue Type: New Feature > Components: server >Reporter: Patrick Hunt >Assignee: Flavio Paiva Junqueira > Fix For: 3.2.0 > > Attachments: ZOOKEEPER-29.patch, ZOOKEEPER-29.patch, > ZOOKEEPER-29.patch, ZOOKEEPER-29.patch, ZOOKEEPER-29.patch, > ZOOKEEPER-29.patch, ZOOKEEPER-29.patch, ZOOKEEPER-29.patch, > ZOOKEEPER-29.patch, ZOOKEEPER-29.patch, ZOOKEEPER-29.patch, > ZOOKEEPER-29.patch, ZOOKEEPER-29.patch, ZOOKEEPER-29.patch, > ZOOKEEPER-29.patch, ZOOKEEPER-29.patch > > > Moved from SourceForge to Apache. > http://sourceforge.net/tracker/index.php?func=detail&aid=1938782&group_id=209147&atid=1008547 -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.
[jira] Updated: (ZOOKEEPER-29) Flexible quorums
[ https://issues.apache.org/jira/browse/ZOOKEEPER-29?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Flavio Paiva Junqueira updated ZOOKEEPER-29: Status: Patch Available (was: Open) > Flexible quorums > > > Key: ZOOKEEPER-29 > URL: https://issues.apache.org/jira/browse/ZOOKEEPER-29 > Project: Zookeeper > Issue Type: New Feature > Components: server >Reporter: Patrick Hunt >Assignee: Flavio Paiva Junqueira > Fix For: 3.2.0 > > Attachments: ZOOKEEPER-29.patch, ZOOKEEPER-29.patch, > ZOOKEEPER-29.patch, ZOOKEEPER-29.patch, ZOOKEEPER-29.patch, > ZOOKEEPER-29.patch, ZOOKEEPER-29.patch, ZOOKEEPER-29.patch, > ZOOKEEPER-29.patch, ZOOKEEPER-29.patch, ZOOKEEPER-29.patch, > ZOOKEEPER-29.patch, ZOOKEEPER-29.patch, ZOOKEEPER-29.patch, > ZOOKEEPER-29.patch, ZOOKEEPER-29.patch > > > Moved from SourceForge to Apache. > http://sourceforge.net/tracker/index.php?func=detail&aid=1938782&group_id=209147&atid=1008547 -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.
[jira] Updated: (ZOOKEEPER-29) Flexible quorums
[ https://issues.apache.org/jira/browse/ZOOKEEPER-29?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Flavio Paiva Junqueira updated ZOOKEEPER-29: Attachment: ZOOKEEPER-29.patch Thanks for the suggestion, Mahadev. I have implemented your suggestion in this new patch. I have also done a couple of other things: 1- I have fixed a bug on FollowerHandler. We should be checking if "qp.getData() == null" instead of checking if "qp.getData().length > 0". With the current protocol, a follower wont't send any data, and qp.getData will be null; 2- I have removed ackCount from the Leader::Proposal class. It is not necessary given that we are keeping a set. All java tests pass for me. > Flexible quorums > > > Key: ZOOKEEPER-29 > URL: https://issues.apache.org/jira/browse/ZOOKEEPER-29 > Project: Zookeeper > Issue Type: New Feature > Components: server >Reporter: Patrick Hunt >Assignee: Flavio Paiva Junqueira > Fix For: 3.2.0 > > Attachments: ZOOKEEPER-29.patch, ZOOKEEPER-29.patch, > ZOOKEEPER-29.patch, ZOOKEEPER-29.patch, ZOOKEEPER-29.patch, > ZOOKEEPER-29.patch, ZOOKEEPER-29.patch, ZOOKEEPER-29.patch, > ZOOKEEPER-29.patch, ZOOKEEPER-29.patch, ZOOKEEPER-29.patch, > ZOOKEEPER-29.patch, ZOOKEEPER-29.patch, ZOOKEEPER-29.patch, > ZOOKEEPER-29.patch, ZOOKEEPER-29.patch > > > Moved from SourceForge to Apache. > http://sourceforge.net/tracker/index.php?func=detail&aid=1938782&group_id=209147&atid=1008547 -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.
[jira] Updated: (ZOOKEEPER-29) Flexible quorums
[ https://issues.apache.org/jira/browse/ZOOKEEPER-29?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Flavio Paiva Junqueira updated ZOOKEEPER-29: Status: Open (was: Patch Available) > Flexible quorums > > > Key: ZOOKEEPER-29 > URL: https://issues.apache.org/jira/browse/ZOOKEEPER-29 > Project: Zookeeper > Issue Type: New Feature > Components: server >Reporter: Patrick Hunt >Assignee: Flavio Paiva Junqueira > Fix For: 3.2.0 > > Attachments: ZOOKEEPER-29.patch, ZOOKEEPER-29.patch, > ZOOKEEPER-29.patch, ZOOKEEPER-29.patch, ZOOKEEPER-29.patch, > ZOOKEEPER-29.patch, ZOOKEEPER-29.patch, ZOOKEEPER-29.patch, > ZOOKEEPER-29.patch, ZOOKEEPER-29.patch, ZOOKEEPER-29.patch, > ZOOKEEPER-29.patch, ZOOKEEPER-29.patch, ZOOKEEPER-29.patch, ZOOKEEPER-29.patch > > > Moved from SourceForge to Apache. > http://sourceforge.net/tracker/index.php?func=detail&aid=1938782&group_id=209147&atid=1008547 -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.
[jira] Commented: (ZOOKEEPER-371) to implement jdiff
[ https://issues.apache.org/jira/browse/ZOOKEEPER-371?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12701451#action_12701451 ] Hadoop QA commented on ZOOKEEPER-371: - -1 overall. Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12406100/zookeeper-371.patch against trunk revision 766897. +1 @author. The patch does not contain any @author tags. +1 tests included. The patch appears to include 1 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 warnings. -1 release audit. The applied patch generated 142 release audit warnings (more than the trunk's current 105 warnings). +1 core tests. The patch passed core unit tests. +1 contrib tests. The patch passed contrib unit tests. Test results: http://hudson.zones.apache.org/hudson/job/Zookeeper-Patch-vesta.apache.org/46/testReport/ Release audit warnings: http://hudson.zones.apache.org/hudson/job/Zookeeper-Patch-vesta.apache.org/46/artifact/trunk/current/releaseAuditDiffWarnings.txt Findbugs warnings: http://hudson.zones.apache.org/hudson/job/Zookeeper-Patch-vesta.apache.org/46/artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html Console output: http://hudson.zones.apache.org/hudson/job/Zookeeper-Patch-vesta.apache.org/46/console This message is automatically generated. > to implement jdiff > --- > > Key: ZOOKEEPER-371 > URL: https://issues.apache.org/jira/browse/ZOOKEEPER-371 > Project: Zookeeper > Issue Type: Improvement > Components: build >Affects Versions: 3.2.0 > Environment: to implement jdiff >Reporter: Giridharan Kesavan >Assignee: Giridharan Kesavan > Fix For: 3.2.0 > > Attachments: zookeeper-371.patch, zookeeper-371.patch > > -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.