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