[jira] Updated: (ZOOKEEPER-622) Test for pending watches in send_set_watches should be moved

2010-03-05 Thread Patrick Hunt (JIRA) 

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

Patrick Hunt updated ZOOKEEPER-622:
---

Status: Open  (was: Patch Available)

 Test for pending watches in send_set_watches should be moved
 

 Key: ZOOKEEPER-622
 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-622
 Project: Zookeeper
  Issue Type: Bug
  Components: c client
Reporter: Steven Cheng
Assignee: Benjamin Reed
 Fix For: 3.3.0

 Attachments: ZOOKEEPER-622.patch, ZOOKEEPER-622.patch, 
 ZOOKEEPER-622.patch, ZOOKEEPER-622.patch


 Valgrind found:
 {quote}
 ==2357== Conditional jump or move depends on uninitialised value(s)
 ==2357==at 0x807FDCA: check_events (zookeeper.c:1180)
 ==2357==by 0x808043A: zookeeper_process (zookeeper.c:1775)
 ==2357==by 0x806A21B: Zookeeper_close::testCloseConnected1() 
 (TestZookeeperClose.cc:161)
 ==2357==by 0x806C6BF: CppUnit::TestCallerZookeeper_close::runTest() 
 (TestCaller.h:166)
 {quote}
 zookeeper.c:1180 was the first if in send_set_watches.

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

[jira] Updated: (ZOOKEEPER-622) Test for pending watches in send_set_watches should be moved

2010-03-05 Thread Patrick Hunt (JIRA) 

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

Patrick Hunt updated ZOOKEEPER-622:
---

Status: Patch Available  (was: Open)

 Test for pending watches in send_set_watches should be moved
 

 Key: ZOOKEEPER-622
 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-622
 Project: Zookeeper
  Issue Type: Bug
  Components: c client
Reporter: Steven Cheng
Assignee: Benjamin Reed
 Fix For: 3.3.0

 Attachments: ZOOKEEPER-622.patch, ZOOKEEPER-622.patch, 
 ZOOKEEPER-622.patch, ZOOKEEPER-622.patch


 Valgrind found:
 {quote}
 ==2357== Conditional jump or move depends on uninitialised value(s)
 ==2357==at 0x807FDCA: check_events (zookeeper.c:1180)
 ==2357==by 0x808043A: zookeeper_process (zookeeper.c:1775)
 ==2357==by 0x806A21B: Zookeeper_close::testCloseConnected1() 
 (TestZookeeperClose.cc:161)
 ==2357==by 0x806C6BF: CppUnit::TestCallerZookeeper_close::runTest() 
 (TestCaller.h:166)
 {quote}
 zookeeper.c:1180 was the first if in send_set_watches.

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

[jira] Updated: (ZOOKEEPER-622) Test for pending watches in send_set_watches should be moved

2010-03-05 Thread Mahadev konar (JIRA) 

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

Mahadev konar updated ZOOKEEPER-622:


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

I just committed this. thanks ben and steven.

 Test for pending watches in send_set_watches should be moved
 

 Key: ZOOKEEPER-622
 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-622
 Project: Zookeeper
  Issue Type: Bug
  Components: c client
Reporter: Steven Cheng
Assignee: Benjamin Reed
 Fix For: 3.3.0

 Attachments: ZOOKEEPER-622.patch, ZOOKEEPER-622.patch, 
 ZOOKEEPER-622.patch, ZOOKEEPER-622.patch


 Valgrind found:
 {quote}
 ==2357== Conditional jump or move depends on uninitialised value(s)
 ==2357==at 0x807FDCA: check_events (zookeeper.c:1180)
 ==2357==by 0x808043A: zookeeper_process (zookeeper.c:1775)
 ==2357==by 0x806A21B: Zookeeper_close::testCloseConnected1() 
 (TestZookeeperClose.cc:161)
 ==2357==by 0x806C6BF: CppUnit::TestCallerZookeeper_close::runTest() 
 (TestCaller.h:166)
 {quote}
 zookeeper.c:1180 was the first if in send_set_watches.

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

[jira] Updated: (ZOOKEEPER-622) Test for pending watches in send_set_watches should be moved

2010-03-03 Thread Benjamin Reed (JIRA) 

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

Benjamin Reed updated ZOOKEEPER-622:


Attachment: ZOOKEEPER-622.patch

this is the patch with just the bug fix in it. the system test has been moved 
to the related jira.

 Test for pending watches in send_set_watches should be moved
 

 Key: ZOOKEEPER-622
 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-622
 Project: Zookeeper
  Issue Type: Bug
  Components: c client
Reporter: Steven Cheng
Assignee: Benjamin Reed
 Fix For: 3.3.0

 Attachments: ZOOKEEPER-622.patch, ZOOKEEPER-622.patch, 
 ZOOKEEPER-622.patch, ZOOKEEPER-622.patch


 Valgrind found:
 {quote}
 ==2357== Conditional jump or move depends on uninitialised value(s)
 ==2357==at 0x807FDCA: check_events (zookeeper.c:1180)
 ==2357==by 0x808043A: zookeeper_process (zookeeper.c:1775)
 ==2357==by 0x806A21B: Zookeeper_close::testCloseConnected1() 
 (TestZookeeperClose.cc:161)
 ==2357==by 0x806C6BF: CppUnit::TestCallerZookeeper_close::runTest() 
 (TestCaller.h:166)
 {quote}
 zookeeper.c:1180 was the first if in send_set_watches.

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

[jira] Updated: (ZOOKEEPER-622) Test for pending watches in send_set_watches should be moved

2010-03-03 Thread Benjamin Reed (JIRA) 

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

Benjamin Reed updated ZOOKEEPER-622:


Status: Patch Available  (was: Open)

 Test for pending watches in send_set_watches should be moved
 

 Key: ZOOKEEPER-622
 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-622
 Project: Zookeeper
  Issue Type: Bug
  Components: c client
Reporter: Steven Cheng
Assignee: Benjamin Reed
 Fix For: 3.3.0

 Attachments: ZOOKEEPER-622.patch, ZOOKEEPER-622.patch, 
 ZOOKEEPER-622.patch, ZOOKEEPER-622.patch


 Valgrind found:
 {quote}
 ==2357== Conditional jump or move depends on uninitialised value(s)
 ==2357==at 0x807FDCA: check_events (zookeeper.c:1180)
 ==2357==by 0x808043A: zookeeper_process (zookeeper.c:1775)
 ==2357==by 0x806A21B: Zookeeper_close::testCloseConnected1() 
 (TestZookeeperClose.cc:161)
 ==2357==by 0x806C6BF: CppUnit::TestCallerZookeeper_close::runTest() 
 (TestCaller.h:166)
 {quote}
 zookeeper.c:1180 was the first if in send_set_watches.

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

[jira] Updated: (ZOOKEEPER-622) Test for pending watches in send_set_watches should be moved

2010-02-23 Thread Steven Cheng (JIRA) 

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

Steven Cheng updated ZOOKEEPER-622:
---

Attachment: ZOOKEEPER-622.patch

A working test case for watch transfer with multiple hosts.  I left testScript 
in there because it is quite useful for basic sanity testing for the scripts.



 Test for pending watches in send_set_watches should be moved
 

 Key: ZOOKEEPER-622
 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-622
 Project: Zookeeper
  Issue Type: Bug
  Components: c client
Reporter: Steven Cheng
Assignee: Benjamin Reed
 Fix For: 3.3.0

 Attachments: ZOOKEEPER-622.patch, ZOOKEEPER-622.patch, 
 ZOOKEEPER-622.patch


 Valgrind found:
 {quote}
 ==2357== Conditional jump or move depends on uninitialised value(s)
 ==2357==at 0x807FDCA: check_events (zookeeper.c:1180)
 ==2357==by 0x808043A: zookeeper_process (zookeeper.c:1775)
 ==2357==by 0x806A21B: Zookeeper_close::testCloseConnected1() 
 (TestZookeeperClose.cc:161)
 ==2357==by 0x806C6BF: CppUnit::TestCallerZookeeper_close::runTest() 
 (TestCaller.h:166)
 {quote}
 zookeeper.c:1180 was the first if in send_set_watches.

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

[jira] Updated: (ZOOKEEPER-622) Test for pending watches in send_set_watches should be moved

2010-01-27 Thread Benjamin Reed (JIRA) 

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

Benjamin Reed updated ZOOKEEPER-622:


Status: Open  (was: Patch Available)

 Test for pending watches in send_set_watches should be moved
 

 Key: ZOOKEEPER-622
 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-622
 Project: Zookeeper
  Issue Type: Bug
  Components: c client
Reporter: Steven Cheng
Assignee: Benjamin Reed
 Fix For: 3.3.0

 Attachments: ZOOKEEPER-622.patch, ZOOKEEPER-622.patch


 Valgrind found:
 {quote}
 ==2357== Conditional jump or move depends on uninitialised value(s)
 ==2357==at 0x807FDCA: check_events (zookeeper.c:1180)
 ==2357==by 0x808043A: zookeeper_process (zookeeper.c:1775)
 ==2357==by 0x806A21B: Zookeeper_close::testCloseConnected1() 
 (TestZookeeperClose.cc:161)
 ==2357==by 0x806C6BF: CppUnit::TestCallerZookeeper_close::runTest() 
 (TestCaller.h:166)
 {quote}
 zookeeper.c:1180 was the first if in send_set_watches.

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

[jira] Updated: (ZOOKEEPER-622) Test for pending watches in send_set_watches should be moved

2009-12-12 Thread Steven Cheng (JIRA) 

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

Steven Cheng updated ZOOKEEPER-622:
---

Attachment: ZOOKEEPER-622.patch

This patch moves the check for outstanding watches in send_set_watches, going 
to try to figure out how to test this later, any suggestions?


 Test for pending watches in send_set_watches should be moved
 

 Key: ZOOKEEPER-622
 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-622
 Project: Zookeeper
  Issue Type: Bug
  Components: c client
Reporter: Steven Cheng
 Fix For: 3.3.0

 Attachments: ZOOKEEPER-622.patch


 Valgrind found:
 {quote}
 ==2357== Conditional jump or move depends on uninitialised value(s)
 ==2357==at 0x807FDCA: check_events (zookeeper.c:1180)
 ==2357==by 0x808043A: zookeeper_process (zookeeper.c:1775)
 ==2357==by 0x806A21B: Zookeeper_close::testCloseConnected1() 
 (TestZookeeperClose.cc:161)
 ==2357==by 0x806C6BF: CppUnit::TestCallerZookeeper_close::runTest() 
 (TestCaller.h:166)
 {quote}
 zookeeper.c:1180 was the first if in send_set_watches.

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