----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46914/ -----------------------------------------------------------
Review request for geode, anilkumar gingade, Barry Oglesby, and nabarun nag. Repository: geode Description ------- LuceneFunction was using sendException to return exceptions to caller. But the behavior of sendException is actually to pass the exception to the addResult method, which is not what we want in this case. Adding an integration test of the same. Changing LuceneFunctionJUnitTest to use mockito and changing the excepectations of what LuceneFunction will do after an exception. Diffs ----- geode-lucene/src/main/java/com/gemstone/gemfire/cache/lucene/internal/distributed/LuceneFunction.java 199b698c295e901cbd5450c263a6301f6db8970c geode-lucene/src/main/java/com/gemstone/gemfire/cache/lucene/internal/distributed/TopEntriesCollectorManager.java b19e1041d830363383c1bfa00bed6f7ac8a4b057 geode-lucene/src/test/java/com/gemstone/gemfire/cache/lucene/LuceneQueriesIntegrationJUnitTest.java PRE-CREATION geode-lucene/src/test/java/com/gemstone/gemfire/cache/lucene/internal/distributed/LuceneFunctionJUnitTest.java 750ec0f6ba2a07e905676900003c24b06647f8ee Diff: https://reviews.apache.org/r/46914/diff/ Testing ------- Thanks, Dan Smith