[OAK-4121] CompactionMap#get not transitive across compaction map generations

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Won't Fix
• Component/s: segmentmk
• Labels: compaction gc
• Epic Link: SegmentMK revision GC
• Assignee: Michael Dürig
• Reporter: Michael Dürig
• Created: 11/Mar/16 14:21
• Updated: 20/Apr/16 13:05
• Resolved: 20/Apr/16 13:05

Description

CompactionMap#get(RecordId before) searches through the compaction maps until it finds one containing before returning its value. However that one might already have been compacted again an be present as key in a later compaction map generation.

A correct implementation of CompactionMap#get(RecordId before) should consider the transitive closure over all maps starting at before. Note however that in this case we would also need to stop removing keys from the compaction map after cleanup as this would break transitivity again. (See http://svn.apache.org/viewvc?view=revision&revision=1673791)

Issue Links

• is blocked by: OAK-4256 CLONE - Cross gc sessions might introduce references to pre-compacted segments
• is blocked by: OAK-3348 Cross gc sessions might introduce references to pre-compacted segments