Benjamin Reed commented on ZOOKEEPER-907:

wow, that is a really dumb error! vishal, could i ask you to modify your patch 
to avoid code duplication?
we should probably have "Request si;" outside of the if, and then just set si 
inside the if statement, and then do the setOwner and submit after the if block 
to avoid code duplication. that way, if we make another change in the future, 
we don't run into this again.

let me know if you need help with the test.

> Spurious "KeeperErrorCode = Session moved" messages
> ---------------------------------------------------
>                 Key: ZOOKEEPER-907
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-907
>             Project: Zookeeper
>          Issue Type: Bug
>    Affects Versions: 3.3.1
>            Reporter: Vishal K
>            Assignee: Vishal K
>            Priority: Blocker
>             Fix For: 3.3.2, 3.4.0
>         Attachments: ZOOKEEPER-907.patch
> The sync request does not set the session owner in Request.
> As a result, the leader keeps printing:
> 2010-07-01 10:55:36,733 - INFO  [ProcessThread:-1:preprequestproces...@405] - 
> Got user-level KeeperException when processing sessionid:0x298d3b1fa90000 
> type:sync: cxid:0x6 zxid:0xfffffffffffffffe txntype:unknown reqpath:/ Error 
> Path:null Error:KeeperErrorCode = Session moved

This message is automatically generated by JIRA.
You can reply to this email to add a comment to the issue online.

Reply via email to