----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39329/ -----------------------------------------------------------
Review request for geode, Ashvin A and xiaojian zhou. Repository: geode Description ------- At the moment, adding an AsyncEventQueue after a region is created has issues - See GEODE-404. Therefore, in order to create a lucene index programmatically, the lucene index needs to be created before the region exists so that we can create the AEQ ahead of time. With this change, when an LuceneIndex is created, the actual creation is deferred in a RegionListener callback. When the region is created, we add the AEQ to the region attributes and create the colocated regions. Diffs ----- gemfire-lucene/src/main/java/com/gemstone/gemfire/cache/lucene/LuceneService.java eed80d97dee539cc3bfb0f9778f957984f3340a7 gemfire-lucene/src/main/java/com/gemstone/gemfire/cache/lucene/internal/LuceneIndexForPartitionedRegion.java 2bf848f6685c21f4199245771c4be3d2a3ba4e83 gemfire-lucene/src/main/java/com/gemstone/gemfire/cache/lucene/internal/LuceneServiceImpl.java 776d005b1091d58c1be46eb9a7a8113ba3daf0d0 gemfire-lucene/src/main/java/com/gemstone/gemfire/cache/lucene/internal/xml/LuceneIndexCreation.java 5520f96242ab02f14e0919144fc9aa3ed4166b92 gemfire-lucene/src/test/java/com/gemstone/gemfire/cache/lucene/internal/LuceneRebalanceJUnitTest.java 478981f56aa4d4eec1a853f85c84014dfd05b5a6 gemfire-lucene/src/test/java/com/gemstone/gemfire/cache/lucene/internal/LuceneServiceImplJUnitTest.java eff28130e2cd03de875b4debf6823a63cded2faa gemfire-lucene/src/test/java/com/gemstone/gemfire/cache/lucene/internal/distributed/LuceneFunctionReadPathDUnitTest.java 6e44b72574d0289f44bbd715ec84409f36fc9aeb gemfire-lucene/src/test/java/com/gemstone/gemfire/cache/lucene/internal/xml/LuceneIndexXmlGeneratorIntegrationJUnitTest.java 65c73f70e69c77bbfa8ef084b1c1712c5d31037b gemfire-lucene/src/test/java/com/gemstone/gemfire/cache/lucene/internal/xml/LuceneIndexXmlParserIntegrationJUnitTest.java 56a726f591f9df0cadd2d4fcf9413abe272e93ea Diff: https://reviews.apache.org/r/39329/diff/ Testing ------- Thanks, Dan Smith
