Re: Possible race in LETest.java

2009-11-11 Thread Flavio Junqueira
Henry already opened one: ZOOKEEPER-569. -Flavio On Nov 11, 2009, at 7:03 AM, Patrick Hunt wrote: Closing the loop - what's the status on this? Can one of you open a JIRA and provide a patch for this? Thanks, Patrick Flavio Junqueira wrote: Hi Henry, Apologies for the the delay. Your

Re: Possible race in LETest.java

2009-11-11 Thread Henry Robinson
The patch is simple, but a test is harder because it's a race condition. I'm working on it. Henry On Wed, Nov 11, 2009 at 12:06 AM, Flavio Junqueira f...@yahoo-inc.comwrote: Henry already opened one: ZOOKEEPER-569. -Flavio On Nov 11, 2009, at 7:03 AM, Patrick Hunt wrote: Closing the

Re: Possible race in LETest.java

2009-11-11 Thread Patrick Hunt
Ok, great. Good to see you guys are on top of this one! Patrick Henry Robinson wrote: The patch is simple, but a test is harder because it's a race condition. I'm working on it. Henry On Wed, Nov 11, 2009 at 12:06 AM, Flavio Junqueira f...@yahoo-inc.comwrote: Henry already opened one:

Re: Possible race in LETest.java

2009-11-10 Thread Patrick Hunt
Closing the loop - what's the status on this? Can one of you open a JIRA and provide a patch for this? Thanks, Patrick Flavio Junqueira wrote: Hi Henry, Apologies for the the delay. Your observation sounds right to me. Here is how I'm reading it; let me know if it makes sense. If everyone

Re: Possible race in LETest.java

2009-10-27 Thread Flavio Junqueira
Hi Henry, I don't understand how 1 and 2 do not end up electing 2 in your situation. If they exclude 3 in countVotes, then countVotes will end up returning 2 and not 3, assuming there is a vote for 2. What am I missing? The problem with QuorumPeer you're pointing at was also an issue with