> On Sept. 1, 2016, 5:26 p.m., Kirk Lund wrote: > > geode-core/src/main/java/com/gemstone/gemfire/internal/SocketCreator.java, > > line 331 > > <https://reviews.apache.org/r/51577/diff/1/?file=1490166#file1490166line331> > > > > No calls ever pass in "true" for reinitialize. I think this shouldn't > > be added unless there's a code path that needs it. Also, we would need a > > new test added that exercises this arg as well. > > Jinmei Liao wrote: > The original calls pass in "true", only the JmxManagerLocatorRequest will > pass in "false"
Oops! I see that now on line 293 of SocketCreator. I closed the issue. - Kirk ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51577/#review147579 ----------------------------------------------------------- On Sept. 1, 2016, 4:30 p.m., Jinmei Liao wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/51577/ > ----------------------------------------------------------- > > (Updated Sept. 1, 2016, 4:30 p.m.) > > > Review request for geode, Bruce Schuchardt and Kirk Lund. > > > Repository: geode > > > Description > ------- > > GEODE-1834: initilize the socketcreator correctly > > > Diffs > ----- > > geode-core/src/main/java/com/gemstone/gemfire/internal/SocketCreator.java > fded3c3b1dc3cf6d2af31291c97adb9f35cd1150 > > geode-core/src/main/java/com/gemstone/gemfire/management/internal/JmxManagerLocatorRequest.java > 861f51d4e0077440ab3128e6053c389cecfd9bb8 > > geode-core/src/test/java/com/gemstone/gemfire/internal/JSSESocketJUnitTest.java > d0d906a8f86ce6a882beb79f91906f5ba5e0ec28 > > Diff: https://reviews.apache.org/r/51577/diff/ > > > Testing > ------- > > > Thanks, > > Jinmei Liao > >