[GEODE-2824] FunctionException: No target node found when executing hasNext on Lucene result

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.2.0
• Component/s: lucene
• Assignee: Xiaojian Zhou
• Reporter: Jason Huynh
• Created: 25/Apr/17 17:11
• Updated: 17/Jun/17 18:03
• Resolved: 08/May/17 17:49

Description

The stack trace below is thrown during a race condition when a node is closing and calling hasNext on a Lucene result.

It looks there was a CacheClosedException, but this execution was unable to find a target node to retry on. This execution then threw a FunctionException.

We have code to unwrap CacheClosedExceptions from function exceptions, however this was just an ordinary function exception. The underlying cause is that the cache is closing at this time.

We should probably wrap all function exceptions with either a LuceneQueryException or equivalent as a user would probably not expect a FunctionException when calling Lucene methods.

The stack trace:

at org.apache.geode.internal.cache.PartitionedRegion.executeOnMultipleNodes(PartitionedRegion.java:3459)
        at org.apache.geode.internal.cache.PartitionedRegion.executeFunction(PartitionedRegion.java:3367)
        at org.apache.geode.internal.cache.execute.PartitionedRegionFunctionExecutor.executeFunction(PartitionedRegionFunctionExecutor.java:228)
        at org.apache.geode.internal.cache.execute.AbstractExecution.execute(AbstractExecution.java:376)
        at org.apache.geode.internal.cache.partitioned.PRFunctionStreamingResultCollector.getResult(PRFunctionStreamingResultCollector.java:178)
        at org.apache.geode.cache.lucene.internal.PageableLuceneQueryResultsImpl.getValues(PageableLuceneQueryResultsImpl.java:112)
        at org.apache.geode.cache.lucene.internal.PageableLuceneQueryResultsImpl.getHitEntries(PageableLuceneQueryResultsImpl.java:91)
        at org.apache.geode.cache.lucene.internal.PageableLuceneQueryResultsImpl.advancePage(PageableLuceneQueryResultsImpl.java:139)
        at org.apache.geode.cache.lucene.internal.PageableLuceneQueryResultsImpl.hasNext(PageableLuceneQueryResultsImpl.java:148)