Marcel Reutegger commented on OAK-4043:

Patch looks good to me. The method in the CheckpointsHelper could be simplified 
using Utils.min(Revision, Revision), but the method was only introduced 
recently (1.4). That is, the current patch is easier to backport to 1.2 and 
earlier if necessary.

> Oak run checkpoints needs to account for multiple index lanes
> -------------------------------------------------------------
>                 Key: OAK-4043
>                 URL: https://issues.apache.org/jira/browse/OAK-4043
>             Project: Jackrabbit Oak
>          Issue Type: Improvement
>          Components: core, run
>            Reporter: Alex Parvulescu
>            Assignee: Alex Parvulescu
>            Priority: Critical
>              Labels: candidate_oak_1_4
>             Fix For: 1.6, 1.5.11
>         Attachments: OAK-4043.patch
> Oak run {{checkpoints rm-unreferenced}} [0] currently is hardcoded on a 
> single checkpoint reference (the default one). Now is it possible to add 
> multiple lanes, which we already did in AEM, but the checkpoint tool is 
> blissfully unaware of this and it might trigger a full reindex following 
> offline compaction.
> fyi [~edivad], [~chetanm]
> [0] https://github.com/apache/jackrabbit-oak/tree/trunk/oak-run#checkpoints

This message was sent by Atlassian JIRA

Reply via email to