Re: Review Request 61758: GEODE-3471: Identify NPE in MBeanProxyFactory

2017-08-21 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61758/#review183399 --- Ship it! Ship It! - Kirk Lund On Aug. 18, 2017, 9:55 p.m., J

Re: Review Request 61758: GEODE-3471: Identify NPE in MBeanProxyFactory

2017-08-21 Thread Patrick Rhomberg
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61758/#review183397 --- Ship it! Ship It! - Patrick Rhomberg On Aug. 18, 2017, 9:55

Re: Review Request 61758: GEODE-3471: Identify NPE in MBeanProxyFactory

2017-08-21 Thread Jared Stewart
> On Aug. 21, 2017, 5:55 p.m., Patrick Rhomberg wrote: > > geode-core/src/main/java/org/apache/geode/management/internal/MBeanProxyFactory.java > > Lines 83-84 (original), 81-85 (patched) > > > > > > I might be missin

Re: Review Request 61758: GEODE-3471: Identify NPE in MBeanProxyFactory

2017-08-21 Thread Patrick Rhomberg
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61758/#review183354 --- geode-core/src/main/java/org/apache/geode/management/internal/MBe

Review Request 61758: GEODE-3471: Identify NPE in MBeanProxyFactory

2017-08-18 Thread Jared Stewart
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61758/ --- Review request for geode, Emily Yeh, Jared Stewart, Ken Howe, Kirk Lund, and Pat