You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@zookeeper.apache.org by "kezhuw (via GitHub)" <gi...@apache.org> on 2023/09/14 12:28:23 UTC

[GitHub] [zookeeper] kezhuw commented on a diff in pull request #1996: ZOOKEEPER-4601: Fix get config watcher path to "/zookeeper/config"

kezhuw commented on code in PR #1996:
URL: https://github.com/apache/zookeeper/pull/1996#discussion_r1325874176


##########
zookeeper-server/src/test/java/org/apache/zookeeper/test/ChrootTest.java:
##########
@@ -63,25 +60,6 @@ public boolean matches() throws InterruptedException {
 
     }
 
-    @Test
-    public void testChrootWithZooKeeperPathWatcher() throws Exception {

Review Comment:
   This test was added in ZOOKEEPER-4565([#1899](https://github.com/apache/zookeeper/pull/1899/files#diff-0174d89616ad3f1405bfcfbbbd6f6f47a24f5f9800ca2a5ace46d67709c7cf1fR66)) which is a predecessor of this deriving issue. I merged it to [`ConfigWatcherPathTest:: testGetConfigWatcherPathWithChroot`](https://github.com/apache/zookeeper/pull/1996/files#diff-74d4f16988f9443ef69492c6f6a479904e8e9eb746564765973cc226015e211bR57) in this pr.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@zookeeper.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org