In getBTreeNode(long page, BTreeNode parent)(line ) if an exception gets thrown for what ever reason it just gets ignored and null returned. This seems problematic because nothing that calls this method checks for null. Worse seems getChildNode(int idx)(line 534) doesn't check and is called in about 11 places and none of their callers check for null.
My hunch is to let the exception be thrown and then we will get meaning full stack traces instead of NPEs later and have to search the log files later. If an exception was thrown like PageNotFound we could catch it in the result code and gracefully skip the document. Thoughts Ideas? Todd Byrne
