Re: Review Request 58742: GEODE-2632: minor fixes from code review

2017-04-27 Thread Ken Howe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58742/#review173224 --- Ship it! Ship It! - Ken Howe On April 26, 2017, 5:17 p.m.,

Re: Review Request 58742: GEODE-2632: minor fixes from code review

2017-04-26 Thread Kirk Lund
> On April 26, 2017, 5:51 p.m., Jinmei Liao wrote: > > geode-core/src/main/java/org/apache/geode/internal/cache/GemFireCacheImpl.java > > Line 2411 (original), 2412 (patched) > > > > > > I know this is the original

Re: Review Request 58742: GEODE-2632: minor fixes from code review

2017-04-26 Thread Patrick Rhomberg
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58742/#review173094 --- Ship it! Ship It! - Patrick Rhomberg On April 26, 2017,

Re: Review Request 58742: GEODE-2632: minor fixes from code review

2017-04-26 Thread Jinmei Liao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58742/#review173066 --- Ship it! Ship It! - Jinmei Liao On April 26, 2017, 5:17

Review Request 58742: GEODE-2632: minor fixes from code review

2017-04-26 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58742/ --- Review request for geode, Jinmei Liao, Jared Stewart, Ken Howe, and Patrick