----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55275/#review160758 -----------------------------------------------------------
Ship it! Ship It! - Barry Oglesby On Jan. 6, 2017, 6:49 p.m., Dan Smith wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/55275/ > ----------------------------------------------------------- > > (Updated Jan. 6, 2017, 6:49 p.m.) > > > Review request for geode and Barry Oglesby. > > > Repository: geode > > > Description > ------- > > If a member without a lucene index is created first, we should throw a > reasonable error rather than an NPE. > > > Diffs > ----- > > geode-core/src/main/java/org/apache/geode/internal/cache/LocalRegion.java > 1343395da972ac5ef07feb424626840b7aea2e99 > > geode-lucene/src/test/java/org/apache/geode/cache/lucene/LuceneIndexCreationDUnitTest.java > 7e50636eaed9bc277f6b25c163695a7c78451610 > > geode-lucene/src/test/java/org/apache/geode/cache/lucene/test/LuceneTestUtilities.java > cb28e88913e9424db075a1b5cf15f4ad89921817 > > Diff: https://reviews.apache.org/r/55275/diff/ > > > Testing > ------- > > > Thanks, > > Dan Smith > >