On Mar 27, 2013, at 10:23 AM, Michi Mutsuzaki <mi...@cs.stanford.edu> wrote: > > Is there a open jira for this patch? I don't have any problem > integrating this patch. Only thing I'd suggest is to not call the > event ZOO_EXPIRED_SESSION_STATE (which should be triggered only by the > server). > > --Michi
I've submitted the JIRA: https://issues.apache.org/jira/browse/ZOOKEEPER-1676 Of note I've submitted links to the patch: https://gist.github.com/yunong/efe869a0345867d54adf as well as my private branch of the patch https://github.com/yunong/zookeeper/tree/release-3.4.3-patched On Mar 27, 2013, at 10:49 AM, Marshall McMullen <marshall.mcmul...@gmail.com> wrote: > We already have a ZOO_NOTCONNECTED_STATE. Seems like that should work for > these purposes… At the time I wrote the patch, I don't think 3.4.3 had ZOO_NOTCONNECTED_STATE. If it's there now in the newer versions, then it certainly seems viable. -Yunong