Re: Review Request 61672: GEODE-3249: internal messages should require credentials

2017-08-17 Thread Bruce Schuchardt
> On Aug. 16, 2017, 4:51 p.m., Brian Rowe wrote: > > geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/ServerConnection.java > > Lines 1059 (patched) > > > > > > Is this intended to be new

Re: Review Request 61672: GEODE-3249: internal messages should require credentials

2017-08-16 Thread Brian Rowe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61672/#review183089 --- Fix it, then Ship it!

Re: Review Request 61672: GEODE-3249: internal messages should require credentials

2017-08-15 Thread Udo Kohlmeyer
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61672/#review182997 --- Ship it! Ship It! - Udo Kohlmeyer On Aug. 15, 2017, 8:46

Review Request 61672: GEODE-3249: internal messages should require credentials

2017-08-15 Thread Bruce Schuchardt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61672/ --- Review request for geode, Alexander Murmann, Galen O'Sullivan, Hitesh Khamesra,