[jira] [Comment Edited] (ZOOKEEPER-1552) Enable sync request processor in Observer

["Mahadev konar (JIRA)" <ji...@apache.org>]

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

Mahadev konar edited comment on ZOOKEEPER-1552 at 12/17/12 6:33 AM:
--------------------------------------------------------------------

Thawan,
 This is a good idea. As for the patch, I think we have too many system properties spread around in the source code. Its best if we can use the ZooKeeper config file for this. What do others think? Other than that, the patch looks good.

                
      was (Author: mahadev):
    Thawan,
 This is a good idea. As for the patch, I think we have too many system properties spread around in the source code. Its best if we can use the ZooKeeper config file for this. What do others think? 

                  
> Enable sync request processor in Observer
> -----------------------------------------
>
>                 Key: ZOOKEEPER-1552
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-1552
>             Project: ZooKeeper
>          Issue Type: Improvement
>          Components: server
>    Affects Versions: 3.4.3
>            Reporter: Thawan Kooburat
>            Assignee: Thawan Kooburat
>             Fix For: 3.5.0
>
>         Attachments: 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