Re: Review Request 62189: GEODE-2817: consolidate authorize(*) methods

2017-09-08 Thread Jinmei Liao
> On Sept. 8, 2017, 10:32 p.m., Ken Howe wrote: > > geode-core/src/main/java/org/apache/geode/cache/execute/Function.java > > Lines 120 (patched) > > > > > > You are expecting an internal object to be passed in a pub

Re: Review Request 62189: GEODE-2817: consolidate authorize(*) methods

2017-09-08 Thread Ken Howe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62189/#review185028 --- geode-core/src/main/java/org/apache/geode/cache/execute/Function.

Re: Review Request 62189: GEODE-2817: consolidate authorize(*) methods

2017-09-08 Thread Jared Stewart
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62189/#review185024 --- Ship it! Ship It! - Jared Stewart On Sept. 8, 2017, 4:28 p.m

Review Request 62189: GEODE-2817: consolidate authorize(*) methods

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