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

Arshad Mohammad commented on ZOOKEEPER-2211:
--------------------------------------------

Thanks a lot  [~rakeshr] for your review comments
# handled
#  Here "greater than 3" means at least 4, but as per the existing 
functionality even 3 snap/logs can be retained.  So I think message "greater 
than 3" will be misleading. IMHO,  "greater than or equal to 3" will be more 
appropriate message. Changed the message to {code}System.out.println("\tcount 
-- the number of old snaps/logs you want to keep, value should be greater than 
or equal to 3");{code} 
is this OK?

> PurgeTxnLog does not correctly purge when snapshots and logs are at different 
> locations
> ---------------------------------------------------------------------------------------
>
>                 Key: ZOOKEEPER-2211
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-2211
>             Project: ZooKeeper
>          Issue Type: Bug
>          Components: scripts
>    Affects Versions: 3.4.6, 3.5.0
>         Environment: Ubuntu 12.04, Java 1.7.
>            Reporter: Wesley Chow
>            Assignee: Arshad Mohammad
>             Fix For: 3.4.7, 3.5.2, 3.6.0
>
>         Attachments: ZOOKEEPER-2211-02.patch, ZOOKEEPER-2211.patch
>
>
> PurgeTxnLog does not work when snapshots and transaction logs are at 
> different file paths. The argument handling is buggy and only works when both 
> snap and datalog dirs are given, and datalog dir contains both logs and snaps 
> (snap is ignored).



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to