----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38513/ -----------------------------------------------------------
Review request for geode and Dan Smith. Repository: geode Description ------- Lucene query distribution depends on ability to serialize and distribute Lucene Query object. For advanced query objects, user can provide a query provider which is expected to take care of query serialization. Add a default implementation based on valid lucene query string for the provider. GEODE-11: Use QueryProvider in LuceneFunction LuceneFunction needs query object for execution. If the argement is missing, fail the function execution Diffs ----- gemfire-lucene/build.gradle 53d47307204760a6875316e44c7b83d024ce0071 gemfire-lucene/src/main/java/com/gemstone/gemfire/cache/lucene/LuceneQueryFactory.java 5636ea4e130d37d00eedf8e66be7cc8c48cb29d2 gemfire-lucene/src/main/java/com/gemstone/gemfire/cache/lucene/LuceneQueryProvider.java PRE-CREATION gemfire-lucene/src/main/java/com/gemstone/gemfire/cache/lucene/internal/LuceneQueryFactoryImpl.java 3bd5b0ccae8b29bcd6b46c316c7f4d02f35d17b9 gemfire-lucene/src/main/java/com/gemstone/gemfire/cache/lucene/internal/StringQueryProvider.java PRE-CREATION gemfire-lucene/src/main/java/com/gemstone/gemfire/cache/lucene/internal/distributed/LuceneFunction.java 25b74540b7dcac6fd2b7afb20062ea2144165b6e gemfire-lucene/src/main/java/com/gemstone/gemfire/cache/lucene/internal/distributed/LuceneFunctionContext.java b3a613fc230e21d6abfaa60258d9bb268be21333 gemfire-lucene/src/test/java/com/gemstone/gemfire/cache/lucene/internal/StringQueryProviderJUnitTest.java PRE-CREATION gemfire-lucene/src/test/java/com/gemstone/gemfire/cache/lucene/internal/distributed/LuceneFunctionJUnitTest.java ec7cd3defa43aa28fc81c90b49d02e4ab9a62259 gemfire-lucene/src/test/java/com/gemstone/gemfire/cache/lucene/internal/distributed/TopEntriesFunctionCollectorJUnitTest.java 6e02e4e2aeb15f196508fe8ec91237a8062d1556 Diff: https://reviews.apache.org/r/38513/diff/ Testing ------- Thanks, Ashvin A
