org.apache.xindice.core.filer.BTree

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