zookeeper-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From anmolnar <...@git.apache.org>
Subject [GitHub] zookeeper issue #377: [ZOOKEEPER-2901] TTL Nodes don't work with Server IDs ...
Date Wed, 09 May 2018 18:29:56 GMT
Github user anmolnar commented on the issue:

    https://github.com/apache/zookeeper/pull/377
  
    @Randgalt Not strictly part of this PR, but I noticed that ContainerManager doesn't log
the name of the container being deleted here:
    
    ```java
    try {
          LOG.info("Attempting to delete candidate container: %s",
                            containerPath);
          requestProcessor.processRequest(request);
    } catch (Exception e) {
           LOG.error(String.format("Could not delete container: %s" ,
                          containerPath), e);
    }
    ```
    
    The `%s` should be `{}`.
    



---

Mime
View raw message