----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59098/#review174346 -----------------------------------------------------------
Ship it! Ship It! - Kirk Lund On May 9, 2017, 3:32 p.m., Jinmei Liao wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/59098/ > ----------------------------------------------------------- > > (Updated May 9, 2017, 3:32 p.m.) > > > Review request for geode, Jared Stewart, Ken Howe, Kirk Lund, and Patrick > Rhomberg. > > > Repository: geode > > > Description > ------- > > GEODE-2876: add logging to diagnose test failure > > another try to add parser logs using log4j. I have another set of changes > that would remove the other loggers in GfshParser. For now, they co-exist. > > > Diffs > ----- > > > geode-core/src/main/java/org/apache/geode/management/internal/cli/GfshParser.java > 20ae0226db45574028311493a07cdfcf9fbfb3f0 > > > Diff: https://reviews.apache.org/r/59098/diff/1/ > > > Testing > ------- > > precheckin running > > > Thanks, > > Jinmei Liao > >
