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

Posted to dev@zookeeper.apache.org by anmolnar <gi...@git.apache.org> on 2018/05/09 22:43:30 UTC

[GitHub] zookeeper pull request #516: ZOOKEEPER-3038 Cleanup some nitpicks in TTL imp...

GitHub user anmolnar opened a pull request:

    https://github.com/apache/zookeeper/pull/516

    ZOOKEEPER-3038 Cleanup some nitpicks in TTL implementation

    A few nitpicks which needs to be cleaned up:
    
    1. Rename OldEphemeralType --> EphemeralTypeEmulate353
    2. Remove unused method: getTTL()
    3. Remove unused import from QuorumPeer


You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/anmolnar/zookeeper ZOOKEEPER-3038

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/zookeeper/pull/516.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #516
    
----
commit 0ba34b35a8fd9c063ca89c1c5697378c608671a1
Author: Andor Molnar <an...@...>
Date:   2018-05-09T22:42:32Z

    ZOOKEEPER-3038. Code review fixes detailed in the Jira

----


---

[GitHub] zookeeper pull request #516: ZOOKEEPER-3038 Cleanup some nitpicks in TTL imp...

Posted by asfgit <gi...@git.apache.org>.

Github user asfgit closed the pull request at:

    https://github.com/apache/zookeeper/pull/516


---

[GitHub] zookeeper issue #516: ZOOKEEPER-3038 Cleanup some nitpicks in TTL implementa...

Posted by phunt <gi...@git.apache.org>.

Github user phunt commented on the issue:

    https://github.com/apache/zookeeper/pull/516
  
    +1 - thanks Andor.


---