So you end up with 2 initialized events?
You mean this?
assertEvent(TreeCacheEvent.Type.CONNECTION_RECONNECTED);
+ assertEvent(TreeCacheEvent.Type.INITIALIZED);
assertEvent(TreeCacheEvent.Type.NODE_REMOVED, "/test/me",
"data".getBytes());
assertEvent(TreeCacheEvent.Type.INITIALIZED);
Seems weird if there are two, but I can help look.
On Thu, Feb 4, 2016 at 10:48 PM, Jordan Zimmerman <
[email protected]> wrote:
> Hey Scott,
>
> In this branch, TestTreeCache.testKilledSession() is failing at:
>
> assertEvent(TreeCacheEvent.Type.NODE_REMOVED, "/test/me",
> "data".getBytes());
>
> However, if I change the two asserts to:
>
> assertEvent(TreeCacheEvent.Type.INITIALIZED);
> assertEvent(TreeCacheEvent.Type.NODE_REMOVED, "/test/me",
> "data".getBytes());
>
> it works. Does that make any sense?
>
> -Jordan
>
> > On Feb 4, 2016, at 9:23 PM, Jordan Zimmerman <[email protected]>
> wrote:
> >
> > Devs,
> >
> > In trying to fix the bad log message "Failed to find watcher” (which
> turns out to be a ZK client issue), I realize that the NamespaceWatcher and
> WatcherWrapper stuff could be improved. I’m still working on getting all
> tests to pass but I’d appreciate more sets of eyes on this change. Please
> review carefully if you can.
> >
> > https://github.com/apache/curator/pull/131
> >
> > -Jordan
>
>
