zookeeper-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From nerdyyatrice <...@git.apache.org>
Subject [GitHub] zookeeper pull request #167: commitProcessor does not crash when an unseen c...
Date Fri, 10 Feb 2017 01:37:29 GMT
GitHub user nerdyyatrice opened a pull request:

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

    commitProcessor does not crash when an unseen commit somes

    commitProcessor with the zookeeper-2024 improvement patch throws an exception when it
sees a commit request that is not at the queue head.  It turned out that it is actually a
valid case when there is session movement. After discussion with the community, I submit this
pull request to mitigate this issue by passing those commits to the next processor instead.

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

    $ git pull https://github.com/nerdyyatrice/zookeeper zookeeper-2684

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

    https://github.com/apache/zookeeper/pull/167.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 #167
    
----
commit 423e385038d055b034a71e91a503ff31532e84a2
Author: rzhang <rzhang@twitter.com>
Date:   2017-02-10T01:33:23Z

    commitProcessor does not crash when an unseen commit somes

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

Mime
View raw message