You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@geode.apache.org by Ashvin A <aa...@gmail.com> on 2015/09/19 01:30:40 UTC
Review Request 38513: GEODE-11: Add LuceneQueryProvider

-----------------------------------------------------------
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