Re: Review Request 60125: GEODE-3062: replace SecurityService after loading cluster config [FIXED]

2017-06-16 Thread Jinmei Liao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60125/#review178114 --- Ship it! Ship It! - Jinmei Liao On June 16, 2017, 4:23

Re: Review Request 60125: GEODE-3062: replace SecurityService after loading cluster config [FIXED]

2017-06-16 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60125/ --- (Updated June 16, 2017, 4:23 p.m.) Review request for geode, Emily Yeh, Jinmei

Re: Review Request 60125: GEODE-3062: replace SecurityService after loading cluster config [FIXED]

2017-06-16 Thread Kirk Lund
> On June 15, 2017, 11:47 p.m., Jinmei Liao wrote: > > geode-core/src/test/java/org/apache/geode/test/dunit/rules/ServerStarterRule.java > > Lines 132 (patched) > > > > > > I am surprised that server.stop() does not

Re: Review Request 60125: GEODE-3062: replace SecurityService after loading cluster config [FIXED]

2017-06-15 Thread Jinmei Liao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60125/#review178082 ---

Re: Review Request 60125: GEODE-3062: replace SecurityService after loading cluster config [FIXED]

2017-06-15 Thread Jinmei Liao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60125/#review178065 ---

Review Request 60125: GEODE-3062: replace SecurityService after loading cluster config [FIXED]

2017-06-15 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60125/ --- Review request for geode, Emily Yeh, Jinmei Liao, Ken Howe, and Patrick