-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45336/
-----------------------------------------------------------
(Updated April 1, 2016, 10:11 p.m.)
Review request for geode, anilkumar gingade and Dan Smith.
Changes
-------
fix based on anil's comments
Bugs: GEM-454, GEM-525 and GEODE-920
https://issues.apache.org/jira/browse/GEM-454
https://issues.apache.org/jira/browse/GEM-525
https://issues.apache.org/jira/browse/GEODE-920
Repository: geode
Description
-------
It's both GEODE-920, GEM-454, GEM-525.
If gateway receiver is created before cache server, the notify subscriptions
attributes can never been set, since CCN singleton is created by the gateway
receiver.
The easy fix is not to let gateway receiver create the CCN singleton.
Diffs (updated)
-----
geode-core/src/main/java/com/gemstone/gemfire/internal/cache/InitialImageOperation.java
9bd3faf
geode-core/src/main/java/com/gemstone/gemfire/internal/cache/LocalRegion.java
8e30a7a
geode-core/src/main/java/com/gemstone/gemfire/internal/cache/ha/HAContainerRegion.java
3bed769
geode-core/src/main/java/com/gemstone/gemfire/internal/cache/tier/sockets/CacheClientNotifier.java
6ac4690
geode-wan/src/test/java/com/gemstone/gemfire/internal/cache/wan/CacheClientNotifierDUnitTest.java
PRE-CREATION
Diff: https://reviews.apache.org/r/45336/diff/
Testing
-------
- introduced new dunit test.
- precheckin
- regression tests:
Thanks,
xiaojian zhou
