----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59893/#review177235 -----------------------------------------------------------
Ship it! To document an offline discussion: Change looks OK with the added checks to verify all expected members are present. However, it wouild be better to eliminate the casue of the "spurious" error. - Ken Howe On June 7, 2017, 9:08 p.m., Jared Stewart wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/59893/ > ----------------------------------------------------------- > > (Updated June 7, 2017, 9:08 p.m.) > > > Review request for geode, Emily Yeh, Jared Stewart, Ken Howe, Kirk Lund, and > Patrick Rhomberg. > > > Repository: geode > > > Description > ------- > > GEODE-3032: Fix CI failure of CommandOverHttpDUnitTest > > > Diffs > ----- > > > geode-core/src/test/java/org/apache/geode/management/internal/cli/commands/MemberCommandsDUnitTest.java > 5bbfc5b3d4ba1a37f8c1e700a36bd96b4ea40667 > > > Diff: https://reviews.apache.org/r/59893/diff/1/ > > > Testing > ------- > > Precheckin is green except for unrelated flaky DUnits. > > > Thanks, > > Jared Stewart > >
