-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45336/
-----------------------------------------------------------
(Updated April 1, 2016, 12:37 a.m.)
Review request for geode, anilkumar gingade and Dan Smith.
Changes
-------
This is a new fix based on a new idea.
The CCN will be created by even the gateway receiver. However, the haContainer
will be kept null and wait for real cache server to initialize it.
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
95616fe
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/AcceptorImpl.java
5b20e86
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
