----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54917/#review159783 -----------------------------------------------------------
I don't think we should mark this test flaky, I think we should fix the product. I tracked this issue down a bit further, it looks like there is a product bug where we can fail to load from cluster configuration. See the ticket for more details. - Dan Smith On Dec. 20, 2016, 10:46 p.m., Kirk Lund wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/54917/ > ----------------------------------------------------------- > > (Updated Dec. 20, 2016, 10:46 p.m.) > > > Review request for geode, Jason Huynh and Dan Smith. > > > Bugs: GEODE-2238 > https://issues.apache.org/jira/browse/GEODE-2238 > > > Repository: geode > > > Description > ------- > > GEODE-2238: add FlakyTest category to flaky test > > We've been seeing this test fail quite frequently in precheckin. I'm adding > FlakyTest category to the test. > > > Diffs > ----- > > > geode-lucene/src/test/java/org/apache/geode/cache/lucene/internal/configuration/LuceneClusterConfigurationDUnitTest.java > 0b0e936652444f0171e37fa52653f5a4e9a63c87 > > Diff: https://reviews.apache.org/r/54917/diff/ > > > Testing > ------- > > > Thanks, > > Kirk Lund > >
