----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39329/#review102811 -----------------------------------------------------------
Ship it! Besides the comments, others look good. gemfire-lucene/src/main/java/com/gemstone/gemfire/cache/lucene/internal/xml/LuceneIndexCreation.java (line 78) <https://reviews.apache.org/r/39329/#comment160582> You assumed the region to be PR here, because only PR has getAttributesMutator().addAEQid(). It's ok, but we need to add a TODO for DR. - xiaojian zhou On Oct. 14, 2015, 9:21 p.m., Dan Smith wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/39329/ > ----------------------------------------------------------- > > (Updated Oct. 14, 2015, 9:21 p.m.) > > > 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 > >