---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/49481/#review140394
---
Looks good! I'm not sure how a feel about adding passing tests that assert that
lucene is broken - if we upgrade lucene those tests might fail, right? Maybe it
would be better to separate out your string query tests into separate tests
that fail, and then mark those tests with @Ingore. That might be better
documentation of what you expect to happen but currently does not work.
- Dan Smith
On June 30, 2016, 11:53 p.m., xiaojian zhou wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/49481/
> ---
>
> (Updated June 30, 2016, 11:53 p.m.)
>
>
> Review request for geode, anilkumar gingade and Dan Smith.
>
>
> Bugs: GEODE-11
> https://issues.apache.org/jira/browse/GEODE-11
>
>
> Repository: geode
>
>
> Description
> ---
>
> The code case archived following conclusions related with research on range
> query
> 1) lucene's QueryParser does not work for numeric range query
> 2) Need a IntPoint.newRangeQuery to do numeric range query for int field.
> Similar for float, double and long fields.
> 3) If field has 2 values with different types, lucene will throw IAE.
> 4) Define a sample implementation for LuceneQueryProvider. This can also be
> used for demo or sample code for customer in the future.
>
>
> Diffs
> -
>
>
> geode-lucene/src/main/java/com/gemstone/gemfire/cache/lucene/internal/LuceneQueryFactoryImpl.java
> 62b4f56
>
> geode-lucene/src/test/java/com/gemstone/gemfire/cache/lucene/LuceneQueriesIntegrationTest.java
> 988e4f5
>
> Diff: https://reviews.apache.org/r/49481/diff/
>
>
> Testing
> ---
>
>
> Thanks,
>
> xiaojian zhou
>
>