----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56425/#review164702 -----------------------------------------------------------
`File.getParentFile()` may return null. For example, assume your working directory is "/tmp". The call to `File(".").getParentFile().getParentFile()` will return null. - Kevin Duling On Feb. 7, 2017, 7:03 p.m., Jinmei Liao wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/56425/ > ----------------------------------------------------------- > > (Updated Feb. 7, 2017, 7:03 p.m.) > > > Review request for geode, Jared Stewart, Kevin Duling, and Kirk Lund. > > > Repository: geode > > > Description > ------- > > GEODE-2272: do not use a new method to start locator with pulse > > > Diffs > ----- > > > geode-assembly/src/test/java/org/apache/geode/tools/pulse/PulseDataExportTest.java > b5472909eec8d5ca124e7bfd5c6cb71864d9bbee > > geode-core/src/test/java/org/apache/geode/test/dunit/rules/LocatorServerStartupRule.java > 1f0cd9e720e732c2ce06515c16601e1df173ff4f > > Diff: https://reviews.apache.org/r/56425/diff/ > > > Testing > ------- > > > Thanks, > > Jinmei Liao > >