[jira] Updated: (ZOOKEEPER-682) Event is not processed when the watcher is set to watch "/" if chrooted
[ https://issues.apache.org/jira/browse/ZOOKEEPER-682?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mahadev konar updated ZOOKEEPER-682: Resolution: Fixed Hadoop Flags: [Reviewed] Status: Resolved (was: Patch Available) I just committed this. thanks scott! > Event is not processed when the watcher is set to watch "/" if chrooted > --- > > Key: ZOOKEEPER-682 > URL: https://issues.apache.org/jira/browse/ZOOKEEPER-682 > Project: Zookeeper > Issue Type: Bug > Components: java client >Affects Versions: 3.2.2 >Reporter: Scott Wang >Assignee: Scott Wang >Priority: Blocker > Fix For: 3.3.0 > > Attachments: ZOOKEEPER-682.patch, ZOOKEEPER-682.patch > > > After the event notification response from server is received, the client > will convert the server path to the client path if chrooted by: > event.setPath(serverPath.substring(chrootPath.length()); > If chrootPath and serverPath are the same, then the event's path will be set > to a null string. > But the key of the watcher's map is "/", not a null string, so the watcher > will not get notified at all. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.
[jira] Updated: (ZOOKEEPER-682) Event is not processed when the watcher is set to watch "/" if chrooted
[ https://issues.apache.org/jira/browse/ZOOKEEPER-682?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Patrick Hunt updated ZOOKEEPER-682: --- Status: Patch Available (was: Open) Thanks Scott. I looked at the c tests and it looks to me like we are testing this case (reviewer please verify). Submitting, let's see how hudson does with this. > Event is not processed when the watcher is set to watch "/" if chrooted > --- > > Key: ZOOKEEPER-682 > URL: https://issues.apache.org/jira/browse/ZOOKEEPER-682 > Project: Zookeeper > Issue Type: Bug > Components: java client >Affects Versions: 3.2.2 >Reporter: Scott Wang >Assignee: Scott Wang >Priority: Blocker > Fix For: 3.3.0 > > Attachments: ZOOKEEPER-682.patch, ZOOKEEPER-682.patch > > > After the event notification response from server is received, the client > will convert the server path to the client path if chrooted by: > event.setPath(serverPath.substring(chrootPath.length()); > If chrootPath and serverPath are the same, then the event's path will be set > to a null string. > But the key of the watcher's map is "/", not a null string, so the watcher > will not get notified at all. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.
[jira] Updated: (ZOOKEEPER-682) Event is not processed when the watcher is set to watch "/" if chrooted
[ https://issues.apache.org/jira/browse/ZOOKEEPER-682?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Scott Wang updated ZOOKEEPER-682: - Attachment: ZOOKEEPER-682.patch Java test case related to this problem is added to the patch attached. The code provides the similar functionality in the c client is investigated: The sub_string function checks if server_path and chroot path are the same and returns "/" if they are the same, so it seems that this problem does not occur on the c client. However, I don't know how to add the test case of this problem to the test cases in c. Anyone can help? > Event is not processed when the watcher is set to watch "/" if chrooted > --- > > Key: ZOOKEEPER-682 > URL: https://issues.apache.org/jira/browse/ZOOKEEPER-682 > Project: Zookeeper > Issue Type: Bug > Components: java client >Affects Versions: 3.2.2 >Reporter: Scott Wang >Assignee: Scott Wang >Priority: Blocker > Fix For: 3.3.0 > > Attachments: ZOOKEEPER-682.patch, ZOOKEEPER-682.patch > > > After the event notification response from server is received, the client > will convert the server path to the client path if chrooted by: > event.setPath(serverPath.substring(chrootPath.length()); > If chrootPath and serverPath are the same, then the event's path will be set > to a null string. > But the key of the watcher's map is "/", not a null string, so the watcher > will not get notified at all. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.
[jira] Updated: (ZOOKEEPER-682) Event is not processed when the watcher is set to watch "/" if chrooted
[ https://issues.apache.org/jira/browse/ZOOKEEPER-682?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Patrick Hunt updated ZOOKEEPER-682: --- Priority: Blocker (was: Major) Fix Version/s: 3.3.0 Assignee: Scott Wang Blocker for 3.3.0, we should ensure that the c client doesn't have similar issues. > Event is not processed when the watcher is set to watch "/" if chrooted > --- > > Key: ZOOKEEPER-682 > URL: https://issues.apache.org/jira/browse/ZOOKEEPER-682 > Project: Zookeeper > Issue Type: Bug > Components: java client >Affects Versions: 3.2.2 >Reporter: Scott Wang >Assignee: Scott Wang >Priority: Blocker > Fix For: 3.3.0 > > Attachments: ZOOKEEPER-682.patch > > > After the event notification response from server is received, the client > will convert the server path to the client path if chrooted by: > event.setPath(serverPath.substring(chrootPath.length()); > If chrootPath and serverPath are the same, then the event's path will be set > to a null string. > But the key of the watcher's map is "/", not a null string, so the watcher > will not get notified at all. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.
[jira] Updated: (ZOOKEEPER-682) Event is not processed when the watcher is set to watch "/" if chrooted
[ https://issues.apache.org/jira/browse/ZOOKEEPER-682?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Scott Wang updated ZOOKEEPER-682: - Attachment: ZOOKEEPER-682.patch Proposed patch attached > Event is not processed when the watcher is set to watch "/" if chrooted > --- > > Key: ZOOKEEPER-682 > URL: https://issues.apache.org/jira/browse/ZOOKEEPER-682 > Project: Zookeeper > Issue Type: Bug > Components: java client >Affects Versions: 3.2.2 >Reporter: Scott Wang > Attachments: ZOOKEEPER-682.patch > > > After the event notification response from server is received, the client > will convert the server path to the client path if chrooted by: > event.setPath(serverPath.substring(chrootPath.length()); > If chrootPath and serverPath are the same, then the event's path will be set > to a null string. > But the key of the watcher's map is "/", not a null string, so the watcher > will not get notified at all. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.