> On Dec. 13, 2016, 3:33 p.m., Jinmei Liao wrote: > > geode-core/src/test/java/org/apache/geode/internal/logging/log4j/custom/CustomConfiguration.java, > > line 60 > > <https://reviews.apache.org/r/54586/diff/1/?file=1580434#file1580434line60> > > > > Looks like this method is never used in the project. We can either > > delete it or refactor the previous method to call this method. > > Kai Jiang wrote: > Okay, got it.
Deleted this. - Kai ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54586/#review159075 ----------------------------------------------------------- On Dec. 19, 2016, 4:45 a.m., Kai Jiang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/54586/ > ----------------------------------------------------------- > > (Updated Dec. 19, 2016, 4:45 a.m.) > > > Review request for geode, Jinmei Liao, Kirk Lund, and Dan Smith. > > > Bugs: geode-2172 > https://issues.apache.org/jira/browse/geode-2172 > > > Repository: geode > > > Description > ------- > > cacheLogWriterMessageShouldMatchCustomConfig fails with AssertionError. > > The issues is related to incorrect regex pattern. The regex pattern used to > match log string contains newline character. To fix this issue, I've replaced > Unix newline character in regex pattern into `System.lineSeparator()`, a > Platform-dependent newline character in Java. > > > Diffs > ----- > > > geode-core/src/test/java/org/apache/geode/internal/logging/log4j/custom/CustomConfiguration.java > 47515e6 > > Diff: https://reviews.apache.org/r/54586/diff/ > > > Testing > ------- > > I've run this unit test both on Windows and MacOS. And it passes. > > > Thanks, > > Kai Jiang > >
