[OAK-3247] DocumentNodeStore.retrieve() should not throw IllegalArgumentException

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 1.3.3
• Fix Version/s: 1.3.5, 1.4
• Component/s: core, mongomk, rdbmk
• Labels: resilience
• Epic Link: DocumentMK resilience
• Assignee: Marcel Reutegger
• Reporter: Julian Sedding
• Created: 18/Aug/15 07:32
• Updated: 02/Mar/16 12:10
• Resolved: 25/Aug/15 13:12

Description

DocumentNodeSTore#retrieve(checkpoint) may throw an IllegalArgumentException via Revision.fromString(checkpoint).

The javadocs say that it returns a NodeState or null. The exception prevents recovery of AsyncIndexUpdate from a bad recorded checkpoint.