GitHub user sijie opened a pull request: https://github.com/apache/incubator-distributedlog/pull/64
DL-91: Be able to close writer within a timeout duration merge twitter's change on closing writer. You can merge this pull request into a Git repository by running: $ git pull https://github.com/sijie/incubator-distributedlog merge/DL-91 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-distributedlog/pull/64.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 #64 ---- 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 <sij...@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 <lstew...@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 <sij...@twitter.com> Date: 2016-05-24T04:01:57Z DL: remove watches when unregister children watches RB_ID=833858 commit 98dc9ab2df8af7af26ca68dbe93be3d420cda417 Author: Sijie Guo <sij...@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 commit d3a97bc0dde0c25516840725599ac46fa03601ab Author: Dave Rusek <dru...@twitter.com> Date: 2016-06-06T23:50:25Z dl: add flag to enable thrift mux on DL Client RB_ID=839555 commit 800b867b705a786094b9a591e0ebc64fa934f632 Author: Dave Rusek <dave.ru...@gmail.com> Date: 2016-12-13T00:41:47Z Don't use stack and codec together for configuring thriftmux - Don't use stack and codec together for configuring thriftmux - Remove codec if the thriftmux is disabled commit 0a18f564ba7e4d03f4c6a0859e5478535a59befd Author: Leigh Stewart <lstew...@twitter.com> Date: 2016-12-13T00:46:11Z Be able to close the writer within a timeout period RB_ID=841340 ---- --- 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. ---