You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@distributedlog.apache.org by sijie <gi...@git.apache.org> on 2016/12/13 02:42:06 UTC

[GitHub] incubator-distributedlog pull request #60: DL-88: remove watches when unregi...

GitHub user sijie opened a pull request:

    https://github.com/apache/incubator-distributedlog/pull/60

    DL-88: remove watches when unregister children watches

    merge twitter's change from Sijie Guo

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

    $ git pull https://github.com/sijie/incubator-distributedlog merge/DL-88

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

    https://github.com/apache/incubator-distributedlog/pull/60.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 #60
    
----
commit 904b8986b46fc908cfd1a0ee05b35dce77d12f5c
Author: Jordan Bull <jb...@twitter.com>
Date:   2016-05-13T18:27:19Z

    when publishing per partition stats, also publish per stream stats
    
    RB_ID=820062

commit 00919605f1bfa3f3e0c8514dc63552df4eeb3cc2
Author: Jordan Bull <jb...@twitter.com>
Date:   2016-05-18T17:18:34Z

    use zero padded partition strings in stats
    
    RB_ID=833803

commit 7b46a9ac6bb5d520366069c244332347ef019e8e
Author: Sijie Guo <si...@twitter.com>
Date:   2016-05-23T23:49:19Z

    Improve handling of lock conflicts in zk session lock
    
    - lock reacquire could happen in foreground and background thread. so use a semaphore to make sure there is only on e outstanding  acquire operation. and  check if already hold lock before reacquire.
    - fix handling zk sibling znode logic. as the znode is sequential znode, the name would be different each time. so only comparing the client id and session id of the znodes
    
    RB_ID=833945

commit 517c77c164cb989ae9829cbd80bf2e492eb8e364
Author: Leigh Stewart <ls...@twitter.com>
Date:   2016-12-13T00:33:33Z

    Introduce periodic keepalive control record in writer
    
        * so the writer will periodically write 'keepalive' control record to make sure the stream is alive. so if the write proxy is disconnected from bookies, the control record will fail to write hence write proxy gets a chance to drop the ownership.

commit b571d3b4adcc140acca881979782474c27459d34
Author: Sijie Guo <si...@twitter.com>
Date:   2016-05-24T04:01:57Z

    DL: remove watches when unregister children watches
    
    RB_ID=833858

commit 98dc9ab2df8af7af26ca68dbe93be3d420cda417
Author: Sijie Guo <si...@twitter.com>
Date:   2016-05-27T00:06:31Z

     Remove watcher only when it registered watcher
    
     (reduce the annoying logging from zookeeper client about "Failed to find watcher!"
    
    RB_ID=837073

----


---
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.
---

[GitHub] incubator-distributedlog pull request #60: DL-88: remove watches when unregi...

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

    https://github.com/apache/incubator-distributedlog/pull/60


---
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.
---