Re: Review Request 47908: GEODE-1460 RemoveAll fails with NPE in com.gemstone.gemfire.internal.cache.tier.sockets.CacheClientNotifier.checkAndRemoveFromClientMsgsRegion()

2016-05-31 Thread Udo Kohlmeyer
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47908/#review135740 --- Ship it! Ship It! - Udo Kohlmeyer On May 26, 2016, 8:34 p.m.

Re: Review Request 47908: GEODE-1460 RemoveAll fails with NPE in com.gemstone.gemfire.internal.cache.tier.sockets.CacheClientNotifier.checkAndRemoveFromClientMsgsRegion()

2016-05-26 Thread Bruce Schuchardt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47908/#review135083 --- Ship it! Ship It! - Bruce Schuchardt On May 26, 2016, 8:34 p

Re: Review Request 47908: GEODE-1460 RemoveAll fails with NPE in com.gemstone.gemfire.internal.cache.tier.sockets.CacheClientNotifier.checkAndRemoveFromClientMsgsRegion()

2016-05-26 Thread xiaojian zhou
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47908/#review135067 --- Ship it! It looks good. Just one qestion, do we need to check

Re: Review Request 47908: GEODE-1460 RemoveAll fails with NPE in com.gemstone.gemfire.internal.cache.tier.sockets.CacheClientNotifier.checkAndRemoveFromClientMsgsRegion()

2016-05-26 Thread Jianxia Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47908/ --- (Updated May 26, 2016, 8:34 p.m.) Review request for geode, anilkumar gingade,

Re: Review Request 47908: GEODE-1460 RemoveAll fails with NPE in com.gemstone.gemfire.internal.cache.tier.sockets.CacheClientNotifier.checkAndRemoveFromClientMsgsRegion()

2016-05-26 Thread Bruce Schuchardt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47908/#review135021 --- I don't understand from your description how this fixes the proble

Re: Review Request 47908: GEODE-1460 RemoveAll fails with NPE in com.gemstone.gemfire.internal.cache.tier.sockets.CacheClientNotifier.checkAndRemoveFromClientMsgsRegion()

2016-05-26 Thread xiaojian zhou
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47908/#review135020 --- geode-core/src/main/java/com/gemstone/gemfire/internal/cache/tier

Review Request 47908: GEODE-1460 RemoveAll fails with NPE in com.gemstone.gemfire.internal.cache.tier.sockets.CacheClientNotifier.checkAndRemoveFromClientMsgsRegion()

2016-05-26 Thread Jianxia Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47908/ --- Review request for geode, anilkumar gingade, Bruce Schuchardt, Hitesh Khamesra,