[jira] Updated: (ZOOKEEPER-221) log message in ZkWatchManager.materialize missing conditional

2008-11-19 Thread Mahadev konar (JIRA) 

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

Mahadev konar updated ZOOKEEPER-221:


Resolution: Fixed
Status: Resolved  (was: Patch Available)

this is fixed in ZOOKEEPER-204.

 log message in ZkWatchManager.materialize missing conditional
 -

 Key: ZOOKEEPER-221
 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-221
 Project: Zookeeper
  Issue Type: Bug
  Components: java client
Affects Versions: 3.0.0
Reporter: Patrick Hunt
Assignee: Patrick Hunt
 Fix For: 3.0.1, 3.1.0

 Attachments: ZOOKEEPER-221.patch


 the following code is missing conditional log call based on result of remove 
 call (size  0)
 // XXX This shouldn't be needed, but just in case
  synchronized (existWatches) {
  addTo(existWatches.remove(path), result);
  LOG.warn(We are triggering an exists watch for delete! Shouldn't 
  happen!);
  }
  

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

[jira] Updated: (ZOOKEEPER-221) log message in ZkWatchManager.materialize missing conditional

2008-11-12 Thread Patrick Hunt (JIRA) 

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

Patrick Hunt updated ZOOKEEPER-221:
---

Status: Patch Available  (was: Open)

 log message in ZkWatchManager.materialize missing conditional
 -

 Key: ZOOKEEPER-221
 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-221
 Project: Zookeeper
  Issue Type: Bug
  Components: java client
Affects Versions: 3.0.0
Reporter: Patrick Hunt
Assignee: Patrick Hunt
 Fix For: 3.0.1, 3.1.0

 Attachments: ZOOKEEPER-221.patch


 the following code is missing conditional log call based on result of remove 
 call (size  0)
 // XXX This shouldn't be needed, but just in case
  synchronized (existWatches) {
  addTo(existWatches.remove(path), result);
  LOG.warn(We are triggering an exists watch for delete! Shouldn't 
  happen!);
  }
  

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

[jira] Updated: (ZOOKEEPER-221) log message in ZkWatchManager.materialize missing conditional

2008-11-12 Thread Patrick Hunt (JIRA) 

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

Patrick Hunt updated ZOOKEEPER-221:
---

Attachment: ZOOKEEPER-221.patch

wrap log message in conditional base on remove result.

 log message in ZkWatchManager.materialize missing conditional
 -

 Key: ZOOKEEPER-221
 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-221
 Project: Zookeeper
  Issue Type: Bug
  Components: java client
Affects Versions: 3.0.0
Reporter: Patrick Hunt
Assignee: Patrick Hunt
 Fix For: 3.0.1, 3.1.0

 Attachments: ZOOKEEPER-221.patch


 the following code is missing conditional log call based on result of remove 
 call (size  0)
 // XXX This shouldn't be needed, but just in case
  synchronized (existWatches) {
  addTo(existWatches.remove(path), result);
  LOG.warn(We are triggering an exists watch for delete! Shouldn't 
  happen!);
  }
  

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