Re: zookeeper_interest returning ZOK on Connection Loss

[Yunong Xiao <yu...@joyent.com>]

On Mar 27, 2013, at 10:49 AM, Marshall McMullen <ma...@gmail.com> wrote:

> We already have a ZOO_NOTCONNECTED_STATE. Seems like that should work for
> these purposes...
> 
> 
> On Wed, Mar 27, 2013 at 11:23 AM, Michi Mutsuzaki <mi...@cs.stanford.edu>wrote:
> 
>> Hi Anthony,
>> 
>> 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
>> 

Michi--

Have you guys had a chance to look at the Jira and patch? I submitted the Jira https://issues.apache.org/jira/browse/ZOOKEEPER-1676 as well a link to my patch https://github.com/yunong/zookeeper/tree/release-3.4.3-patched a couple weeks ago. Just wondered if there's anything else you need from me in order to get this patch integrated.

-Yunong

Re: zookeeper_interest returning ZOK on Connection Loss

[Michi Mutsuzaki <mi...@cs.stanford.edu>]

Hi Yunong,

I haven't had a chance to review your patch yet. I think I'll have
some time this weekend.

Thanks!
--Michi

On Mon, Apr 8, 2013 at 7:14 PM, Yunong Xiao <yu...@joyent.com> wrote:
>
> On Mar 27, 2013, at 10:49 AM, Marshall McMullen
> <ma...@gmail.com> wrote:
>
> We already have a ZOO_NOTCONNECTED_STATE. Seems like that should work for
> these purposes...
>
>
> On Wed, Mar 27, 2013 at 11:23 AM, Michi Mutsuzaki
> <mi...@cs.stanford.edu>wrote:
>
> Hi Anthony,
>
> 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
>
>
> Michi--
>
> Have you guys had a chance to look at the Jira and patch? I submitted the
> Jira https://issues.apache.org/jira/browse/ZOOKEEPER-1676 as well a link to
> my patch https://github.com/yunong/zookeeper/tree/release-3.4.3-patched a
> couple weeks ago. Just wondered if there's anything else you need from me in
> order to get this patch integrated.
>
> -Yunong