Vishal K commented on ZOOKEEPER-872:

Is there any reason why we dont have a findNRecentLogs(int n) method to return 
the n most recent logs (similar to findNRecentSnapshots)?
While testing I noticed that it can happen that a log file is left undeleted 
depending on the transaction id of the nth snapshot file.  Thus, we will have n 
snapshots, but n+1 log file left behind instead of n. This file will be deleted 
after the next snapshot is taken.

We won't have this problem if we have a function that keeps the n most recent 
logs and removes the rest.

> Small fixes to PurgeTxnLog 
> ---------------------------
>                 Key: ZOOKEEPER-872
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-872
>             Project: Zookeeper
>          Issue Type: Bug
>    Affects Versions: 3.3.1
>            Reporter: Vishal K
>            Priority: Minor
>         Attachments: ZOOKEEPER-872
> PurgeTxnLog forces us to have at least 2 backups (by having count >= 3. Also, 
> it prints to stdout instead of using Logger.

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