[jira] Updated: (ZOOKEEPER-693) TestObserver stuck in tight notification loop in FLE

2010-03-10 Thread Patrick Hunt (JIRA)

 [ 
https://issues.apache.org/jira/browse/ZOOKEEPER-693?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Patrick Hunt updated ZOOKEEPER-693:
---

  Resolution: Fixed
Hadoop Flags: [Reviewed]
  Status: Resolved  (was: Patch Available)

+1 looks good, tests pass on my system as well.
Committed revision 921509.


> TestObserver stuck in tight notification loop in FLE
> 
>
> Key: ZOOKEEPER-693
> URL: https://issues.apache.org/jira/browse/ZOOKEEPER-693
> Project: Zookeeper
>  Issue Type: Bug
>Affects Versions: 3.3.0
>Reporter: Henry Robinson
>Assignee: Flavio Paiva Junqueira
>Priority: Critical
> Fix For: 3.3.0
>
> Attachments: ZOOKEEPER-693.patch, ZOOKEEPER-693.patch
>
>
> See 
> http://hudson.zones.apache.org/hudson/view/ZooKeeper/job/Zookeeper-Patch-h7.grid.sp2.yahoo.net/77/testReport/junit/org.apache.zookeeper.test/ObserverTest/testObserver/
>  [exec] [junit] 2010-03-04 00:23:37,803 - INFO  
> [QuorumPeer:/0:0:0:0:0:0:0:0:11229:fastleaderelect...@683] - Notification 
> time out: 3200
>  [exec] [junit] 2010-03-04 00:23:37,804 - INFO  
> [QuorumPeer:/0:0:0:0:0:0:0:0:11229:fastleaderelect...@689] - Notification: 2, 
> 0, 2, 3, LOOKING, LOOKING, 1
> ad infinitum. 

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



[jira] Updated: (ZOOKEEPER-693) TestObserver stuck in tight notification loop in FLE

2010-03-10 Thread Flavio Paiva Junqueira (JIRA)

 [ 
https://issues.apache.org/jira/browse/ZOOKEEPER-693?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Flavio Paiva Junqueira updated ZOOKEEPER-693:
-

Status: Patch Available  (was: Open)

> TestObserver stuck in tight notification loop in FLE
> 
>
> Key: ZOOKEEPER-693
> URL: https://issues.apache.org/jira/browse/ZOOKEEPER-693
> Project: Zookeeper
>  Issue Type: Bug
>Affects Versions: 3.3.0
>Reporter: Henry Robinson
>Assignee: Flavio Paiva Junqueira
>Priority: Critical
> Fix For: 3.3.0
>
> Attachments: ZOOKEEPER-693.patch, ZOOKEEPER-693.patch
>
>
> See 
> http://hudson.zones.apache.org/hudson/view/ZooKeeper/job/Zookeeper-Patch-h7.grid.sp2.yahoo.net/77/testReport/junit/org.apache.zookeeper.test/ObserverTest/testObserver/
>  [exec] [junit] 2010-03-04 00:23:37,803 - INFO  
> [QuorumPeer:/0:0:0:0:0:0:0:0:11229:fastleaderelect...@683] - Notification 
> time out: 3200
>  [exec] [junit] 2010-03-04 00:23:37,804 - INFO  
> [QuorumPeer:/0:0:0:0:0:0:0:0:11229:fastleaderelect...@689] - Notification: 2, 
> 0, 2, 3, LOOKING, LOOKING, 1
> ad infinitum. 

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



[jira] Updated: (ZOOKEEPER-693) TestObserver stuck in tight notification loop in FLE

2010-03-10 Thread Flavio Paiva Junqueira (JIRA)

 [ 
https://issues.apache.org/jira/browse/ZOOKEEPER-693?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Flavio Paiva Junqueira updated ZOOKEEPER-693:
-

Status: Open  (was: Patch Available)

> TestObserver stuck in tight notification loop in FLE
> 
>
> Key: ZOOKEEPER-693
> URL: https://issues.apache.org/jira/browse/ZOOKEEPER-693
> Project: Zookeeper
>  Issue Type: Bug
>Affects Versions: 3.3.0
>Reporter: Henry Robinson
>Assignee: Flavio Paiva Junqueira
>Priority: Critical
> Fix For: 3.3.0
>
> Attachments: ZOOKEEPER-693.patch, ZOOKEEPER-693.patch
>
>
> See 
> http://hudson.zones.apache.org/hudson/view/ZooKeeper/job/Zookeeper-Patch-h7.grid.sp2.yahoo.net/77/testReport/junit/org.apache.zookeeper.test/ObserverTest/testObserver/
>  [exec] [junit] 2010-03-04 00:23:37,803 - INFO  
> [QuorumPeer:/0:0:0:0:0:0:0:0:11229:fastleaderelect...@683] - Notification 
> time out: 3200
>  [exec] [junit] 2010-03-04 00:23:37,804 - INFO  
> [QuorumPeer:/0:0:0:0:0:0:0:0:11229:fastleaderelect...@689] - Notification: 2, 
> 0, 2, 3, LOOKING, LOOKING, 1
> ad infinitum. 

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



[jira] Updated: (ZOOKEEPER-693) TestObserver stuck in tight notification loop in FLE

2010-03-10 Thread Flavio Paiva Junqueira (JIRA)

 [ 
https://issues.apache.org/jira/browse/ZOOKEEPER-693?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Flavio Paiva Junqueira updated ZOOKEEPER-693:
-

Attachment: ZOOKEEPER-693.patch

My bad, forgot to fix AuthFLE as well.

> TestObserver stuck in tight notification loop in FLE
> 
>
> Key: ZOOKEEPER-693
> URL: https://issues.apache.org/jira/browse/ZOOKEEPER-693
> Project: Zookeeper
>  Issue Type: Bug
>Affects Versions: 3.3.0
>Reporter: Henry Robinson
>Assignee: Flavio Paiva Junqueira
>Priority: Critical
> Fix For: 3.3.0
>
> Attachments: ZOOKEEPER-693.patch, ZOOKEEPER-693.patch
>
>
> See 
> http://hudson.zones.apache.org/hudson/view/ZooKeeper/job/Zookeeper-Patch-h7.grid.sp2.yahoo.net/77/testReport/junit/org.apache.zookeeper.test/ObserverTest/testObserver/
>  [exec] [junit] 2010-03-04 00:23:37,803 - INFO  
> [QuorumPeer:/0:0:0:0:0:0:0:0:11229:fastleaderelect...@683] - Notification 
> time out: 3200
>  [exec] [junit] 2010-03-04 00:23:37,804 - INFO  
> [QuorumPeer:/0:0:0:0:0:0:0:0:11229:fastleaderelect...@689] - Notification: 2, 
> 0, 2, 3, LOOKING, LOOKING, 1
> ad infinitum. 

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



[jira] Updated: (ZOOKEEPER-693) TestObserver stuck in tight notification loop in FLE

2010-03-10 Thread Flavio Paiva Junqueira (JIRA)

 [ 
https://issues.apache.org/jira/browse/ZOOKEEPER-693?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Flavio Paiva Junqueira updated ZOOKEEPER-693:
-

Attachment: ZOOKEEPER-693.patch

Taking a look at the log, I realized that that the execution of the two tests 
of ObserverTest were overlapping. More specifically, leader election instances 
were not being shut down correctly after the first test (testObserver), and it 
seems that they were interfering with the second test (testSingleObserver).  

The real issue is that I have been shutting down FLE separately from the 
QuorumPeer in tests, and QuorumPeerTestBase that ObserverTest is using has no 
way to access the underlying QuorumPeer instance. 

To solve this problem, I simply added code to QuorumPeer:shutdown so that we 
make sure that we shutdown all leader election instances. I had to also add a 
new method to the Election interface, and implement an empty method in 
LeaderElection.

Since it is ok to call shutdown multiple times, I'll leave the FLE tests 
untouched for this patch. I have run the tests a few times, and I didn't see 
any issue. I'll create a jira issue to remove the duplicate calls to shutdown. 

> TestObserver stuck in tight notification loop in FLE
> 
>
> Key: ZOOKEEPER-693
> URL: https://issues.apache.org/jira/browse/ZOOKEEPER-693
> Project: Zookeeper
>  Issue Type: Bug
>Affects Versions: 3.3.0
>Reporter: Henry Robinson
>Priority: Critical
> Fix For: 3.3.0
>
> Attachments: ZOOKEEPER-693.patch
>
>
> See 
> http://hudson.zones.apache.org/hudson/view/ZooKeeper/job/Zookeeper-Patch-h7.grid.sp2.yahoo.net/77/testReport/junit/org.apache.zookeeper.test/ObserverTest/testObserver/
>  [exec] [junit] 2010-03-04 00:23:37,803 - INFO  
> [QuorumPeer:/0:0:0:0:0:0:0:0:11229:fastleaderelect...@683] - Notification 
> time out: 3200
>  [exec] [junit] 2010-03-04 00:23:37,804 - INFO  
> [QuorumPeer:/0:0:0:0:0:0:0:0:11229:fastleaderelect...@689] - Notification: 2, 
> 0, 2, 3, LOOKING, LOOKING, 1
> ad infinitum. 

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



[jira] Updated: (ZOOKEEPER-693) TestObserver stuck in tight notification loop in FLE

2010-03-10 Thread Flavio Paiva Junqueira (JIRA)

 [ 
https://issues.apache.org/jira/browse/ZOOKEEPER-693?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Flavio Paiva Junqueira updated ZOOKEEPER-693:
-

Assignee: Flavio Paiva Junqueira
  Status: Patch Available  (was: Open)

> TestObserver stuck in tight notification loop in FLE
> 
>
> Key: ZOOKEEPER-693
> URL: https://issues.apache.org/jira/browse/ZOOKEEPER-693
> Project: Zookeeper
>  Issue Type: Bug
>Affects Versions: 3.3.0
>Reporter: Henry Robinson
>Assignee: Flavio Paiva Junqueira
>Priority: Critical
> Fix For: 3.3.0
>
> Attachments: ZOOKEEPER-693.patch
>
>
> See 
> http://hudson.zones.apache.org/hudson/view/ZooKeeper/job/Zookeeper-Patch-h7.grid.sp2.yahoo.net/77/testReport/junit/org.apache.zookeeper.test/ObserverTest/testObserver/
>  [exec] [junit] 2010-03-04 00:23:37,803 - INFO  
> [QuorumPeer:/0:0:0:0:0:0:0:0:11229:fastleaderelect...@683] - Notification 
> time out: 3200
>  [exec] [junit] 2010-03-04 00:23:37,804 - INFO  
> [QuorumPeer:/0:0:0:0:0:0:0:0:11229:fastleaderelect...@689] - Notification: 2, 
> 0, 2, 3, LOOKING, LOOKING, 1
> ad infinitum. 

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