[ 
https://issues.apache.org/jira/browse/ZOOKEEPER-1552?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13660188#comment-13660188
 ] 

Flavio Junqueira commented on ZOOKEEPER-1552:
---------------------------------------------

My take is that we do not have to be strict and classify this jira as a bug 
just so that we have it in the 3.4 branch. I see this patch as an improvement 
because the way the code currently is is correct, but perhaps not as efficient 
as you'd like. 

I'd like to have a last look at the patch, since I haven't looked at this issue 
in a while. In any case, this is the next patch to get in in the description 
you added to ZOOKEEPER-1551, right? 
                
> Enable sync request processor in Observer
> -----------------------------------------
>
>                 Key: ZOOKEEPER-1552
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-1552
>             Project: ZooKeeper
>          Issue Type: Improvement
>          Components: quorum, server
>    Affects Versions: 3.4.3
>            Reporter: Thawan Kooburat
>            Assignee: Thawan Kooburat
>             Fix For: 3.5.0
>
>         Attachments: ZOOKEEPER-1552.patch, ZOOKEEPER-1552.patch, 
> ZOOKEEPER-1552.patch
>
>
> Observer doesn't forward its txns to SyncRequestProcessor. So it never 
> persists the txns onto disk or periodically creates snapshots. This increases 
> the start-up time since it will get the entire snapshot if the observer has 
> be running for a long time. 

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to