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

[asfgit]

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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[sijie]

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 
Date:   2016-05-13T18:27:19Z

when publishing per partition stats, also publish per stream stats

RB_ID=820062

commit 00919605f1bfa3f3e0c8514dc63552df4eeb3cc2
Author: Jordan Bull 
Date:   2016-05-18T17:18:34Z

use zero padded partition strings in stats

RB_ID=833803

commit 7b46a9ac6bb5d520366069c244332347ef019e8e
Author: Sijie Guo 
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 
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 
Date:   2016-05-24T04:01:57Z

DL: remove watches when unregister children watches

RB_ID=833858

commit 98dc9ab2df8af7af26ca68dbe93be3d420cda417
Author: Sijie Guo 
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---