[OAK-4371] Overly zealous warning about checkpoints on compaction

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: Segment Tar 0.0.14
• Component/s: segment-tar, segmentmk
• Labels: compaction gc logging
• Epic Link: SegmentMK revision GC
• Assignee: Michael Dürig
• Reporter: Michael Dürig
• Created: 17/May/16 09:23
• Updated: 12/Oct/16 06:41
• Resolved: 22/Sep/16 09:23

Description

FileStore.compact logs a warning TarMK GC #{}: compaction found {} checkpoints, you might need to run checkpoint cleanup if there is more than a single checkpoints.

AFIK this is now the norm as async indexing has uses 2 checkpoints (chetanm, edivad please clarify).

In any case should we improve this and not hard code any number of expected checkpoints. Maybe make the threshold configurable?

Issue Links

• is related to: OAK-4927 FileStore compaction should account for multiple valid checkpoints
• relates to: OAK-4043 Oak run checkpoints needs to account for multiple index lanes