You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@geode.apache.org by Jason Huynh <hu...@gmail.com> on 2016/04/09 01:15:41 UTC

Review Request 45947: Update lucene version to 6.0.0

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45947/
-----------------------------------------------------------

Review request for geode, anilkumar gingade, Barry Oglesby, nabarun nag, Dan Smith, and xiaojian zhou.


Repository: geode


Description
-------

Updated version to 6.0.0
Made modifications to use DoublePoint instead of LegacyDoubleField, IntPoint instead of LegacyIntField, etc
Added a temporary file to FileSystem.  This can be discussed as to how we want to handle this method.  Currently it creates a temporary, non distributed file.
SearcherManager creation now will pass in true for force deletes... This can also be discussed on what we want to pass in

Sorting the fileList due to new Lucene based test that requires the list returned to be sorted


Diffs
-----

  geode-lucene/src/main/java/com/gemstone/gemfire/cache/lucene/internal/directory/RegionDirectory.java e25dc77 
  geode-lucene/src/main/java/com/gemstone/gemfire/cache/lucene/internal/filesystem/FileSystem.java b84dc92 
  geode-lucene/src/main/java/com/gemstone/gemfire/cache/lucene/internal/repository/IndexRepositoryImpl.java a9c463e 
  geode-lucene/src/main/java/com/gemstone/gemfire/cache/lucene/internal/repository/serializer/SerializerUtil.java 7ffc5db 
  geode-lucene/src/test/java/com/gemstone/gemfire/cache/lucene/internal/repository/IndexRepositoryImplPerformanceTest.java 74f3742 
  gradle/dependency-versions.properties 7a5054b 

Diff: https://reviews.apache.org/r/45947/diff/


Testing
-------

geode-lucene:precheckin
running precheckin


Thanks,

Jason Huynh