DocumentNodeStore.retrieve() should not throw IllegalArgumentException

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.