[jira] Commented: (ZOOKEEPER-508) proposals and commits for DIFF and Truncate messages from the leader to followers is buggy.

2009-08-24 Thread Benjamin Reed (JIRA)

[ 
https://issues.apache.org/jira/browse/ZOOKEEPER-508?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12747155#action_12747155
 ] 

Benjamin Reed commented on ZOOKEEPER-508:
-

+1 looks good. simple fix! :)

 proposals and commits for DIFF and Truncate messages from the leader to 
 followers is buggy.
 ---

 Key: ZOOKEEPER-508
 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-508
 Project: Zookeeper
  Issue Type: Bug
  Components: quorum
Reporter: Mahadev konar
Assignee: Mahadev konar
Priority: Blocker
 Fix For: 3.2.1, 3.3.0

 Attachments: ZOOKEEPER-508.patch, ZOOKEEPER-508.patch, 
 ZOOKEEPER-508.patch, ZOOKEEPER-508.patch, ZOOKEEPER-508.patch, 
 ZOOKEEPER-508.patch-3.2


 The proposals and commits sent by the leader after it asks the followers to 
 truncate there logs or starts sending a diff has missing messages which 
 causes out of order commits messages and causes the followers to shutdown 
 because of these out of order commits.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Commented: (ZOOKEEPER-508) proposals and commits for DIFF and Truncate messages from the leader to followers is buggy.

2009-08-21 Thread Hadoop QA (JIRA)

[ 
https://issues.apache.org/jira/browse/ZOOKEEPER-508?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12746172#action_12746172
 ] 

Hadoop QA commented on ZOOKEEPER-508:
-

-1 overall.  Here are the results of testing the latest attachment 
  http://issues.apache.org/jira/secure/attachment/12417207/ZOOKEEPER-508.patch
  against trunk revision 803300.

+1 @author.  The patch does not contain any @author tags.

+1 tests included.  The patch appears to include 9 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 failed 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/189/testReport/
Findbugs warnings: 
http://hudson.zones.apache.org/hudson/job/Zookeeper-Patch-vesta.apache.org/189/artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html
Console output: 
http://hudson.zones.apache.org/hudson/job/Zookeeper-Patch-vesta.apache.org/189/console

This message is automatically generated.

 proposals and commits for DIFF and Truncate messages from the leader to 
 followers is buggy.
 ---

 Key: ZOOKEEPER-508
 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-508
 Project: Zookeeper
  Issue Type: Bug
  Components: quorum
Reporter: Mahadev konar
Assignee: Mahadev konar
Priority: Blocker
 Fix For: 3.2.1, 3.3.0

 Attachments: ZOOKEEPER-508.patch, ZOOKEEPER-508.patch, 
 ZOOKEEPER-508.patch, ZOOKEEPER-508.patch


 The proposals and commits sent by the leader after it asks the followers to 
 truncate there logs or starts sending a diff has missing messages which 
 causes out of order commits messages and causes the followers to shutdown 
 because of these out of order commits.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Commented: (ZOOKEEPER-508) proposals and commits for DIFF and Truncate messages from the leader to followers is buggy.

2009-08-21 Thread Hadoop QA (JIRA)

[ 
https://issues.apache.org/jira/browse/ZOOKEEPER-508?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12746400#action_12746400
 ] 

Hadoop QA commented on ZOOKEEPER-508:
-

+1 overall.  Here are the results of testing the latest attachment 
  http://issues.apache.org/jira/secure/attachment/12417339/ZOOKEEPER-508.patch
  against trunk revision 803300.

+1 @author.  The patch does not contain any @author tags.

+1 tests included.  The patch appears to include 9 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/190/testReport/
Findbugs warnings: 
http://hudson.zones.apache.org/hudson/job/Zookeeper-Patch-vesta.apache.org/190/artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html
Console output: 
http://hudson.zones.apache.org/hudson/job/Zookeeper-Patch-vesta.apache.org/190/console

This message is automatically generated.

 proposals and commits for DIFF and Truncate messages from the leader to 
 followers is buggy.
 ---

 Key: ZOOKEEPER-508
 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-508
 Project: Zookeeper
  Issue Type: Bug
  Components: quorum
Reporter: Mahadev konar
Assignee: Mahadev konar
Priority: Blocker
 Fix For: 3.2.1, 3.3.0

 Attachments: ZOOKEEPER-508.patch, ZOOKEEPER-508.patch, 
 ZOOKEEPER-508.patch, ZOOKEEPER-508.patch, ZOOKEEPER-508.patch, 
 ZOOKEEPER-508.patch-3.2


 The proposals and commits sent by the leader after it asks the followers to 
 truncate there logs or starts sending a diff has missing messages which 
 causes out of order commits messages and causes the followers to shutdown 
 because of these out of order commits.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Commented: (ZOOKEEPER-508) proposals and commits for DIFF and Truncate messages from the leader to followers is buggy.

2009-08-20 Thread Mahadev konar (JIRA)

[ 
https://issues.apache.org/jira/browse/ZOOKEEPER-508?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12745683#action_12745683
 ] 

Mahadev konar commented on ZOOKEEPER-508:
-

i fugured out the reason why it fails on the assertion errors != 0. The whole 
scenario of 483 fialing with truncate is this

- the test case shutdowns all the followers
- the leader does not realize that its lost the leadership, becasue the time we 
ping to see if the leader is still the leader (maybe 1 sec) is greater than the 
time hte followers actually take to shutdown and get back and in sync with the 
leader
- so the leader never shutsdown any of its stuff (no NIO rejection or nething 
else)

so in your case, sometimes the client conencts to the leader and will never see 
errors. On the other hand sometimes it may pas s on connection to other 
followers and your testcase will pass. So we cannot really say that your test 
case is fool proof.



 proposals and commits for DIFF and Truncate messages from the leader to 
 followers is buggy.
 ---

 Key: ZOOKEEPER-508
 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-508
 Project: Zookeeper
  Issue Type: Bug
  Components: quorum
Reporter: Mahadev konar
Assignee: Mahadev konar
Priority: Blocker
 Fix For: 3.2.1, 3.3.0

 Attachments: ZOOKEEPER-508.patch, ZOOKEEPER-508.patch


 The proposals and commits sent by the leader after it asks the followers to 
 truncate there logs or starts sending a diff has missing messages which 
 causes out of order commits messages and causes the followers to shutdown 
 because of these out of order commits.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Commented: (ZOOKEEPER-508) proposals and commits for DIFF and Truncate messages from the leader to followers is buggy.

2009-08-20 Thread Mahadev konar (JIRA)

[ 
https://issues.apache.org/jira/browse/ZOOKEEPER-508?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12745733#action_12745733
 ] 

Mahadev konar commented on ZOOKEEPER-508:
-

I have a patch for the 3.2 branch, will upload is as soon as hudson is done 
running this patch.

 proposals and commits for DIFF and Truncate messages from the leader to 
 followers is buggy.
 ---

 Key: ZOOKEEPER-508
 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-508
 Project: Zookeeper
  Issue Type: Bug
  Components: quorum
Reporter: Mahadev konar
Assignee: Mahadev konar
Priority: Blocker
 Fix For: 3.2.1, 3.3.0

 Attachments: ZOOKEEPER-508.patch, ZOOKEEPER-508.patch, 
 ZOOKEEPER-508.patch


 The proposals and commits sent by the leader after it asks the followers to 
 truncate there logs or starts sending a diff has missing messages which 
 causes out of order commits messages and causes the followers to shutdown 
 because of these out of order commits.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Commented: (ZOOKEEPER-508) proposals and commits for DIFF and Truncate messages from the leader to followers is buggy.

2009-08-20 Thread Hadoop QA (JIRA)

[ 
https://issues.apache.org/jira/browse/ZOOKEEPER-508?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12745755#action_12745755
 ] 

Hadoop QA commented on ZOOKEEPER-508:
-

-1 overall.  Here are the results of testing the latest attachment 
  http://issues.apache.org/jira/secure/attachment/12417189/ZOOKEEPER-508.patch
  against trunk revision 803300.

+1 @author.  The patch does not contain any @author tags.

+1 tests included.  The patch appears to include 9 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 178 release audit warnings 
(more than the trunk's current 177 warnings).

-1 core tests.  The patch failed 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/188/testReport/
Release audit warnings: 
http://hudson.zones.apache.org/hudson/job/Zookeeper-Patch-vesta.apache.org/188/artifact/trunk/patchprocess/releaseAuditDiffWarnings.txt
Findbugs warnings: 
http://hudson.zones.apache.org/hudson/job/Zookeeper-Patch-vesta.apache.org/188/artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html
Console output: 
http://hudson.zones.apache.org/hudson/job/Zookeeper-Patch-vesta.apache.org/188/console

This message is automatically generated.

 proposals and commits for DIFF and Truncate messages from the leader to 
 followers is buggy.
 ---

 Key: ZOOKEEPER-508
 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-508
 Project: Zookeeper
  Issue Type: Bug
  Components: quorum
Reporter: Mahadev konar
Assignee: Mahadev konar
Priority: Blocker
 Fix For: 3.2.1, 3.3.0

 Attachments: ZOOKEEPER-508.patch, ZOOKEEPER-508.patch, 
 ZOOKEEPER-508.patch


 The proposals and commits sent by the leader after it asks the followers to 
 truncate there logs or starts sending a diff has missing messages which 
 causes out of order commits messages and causes the followers to shutdown 
 because of these out of order commits.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.