----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46625/ -----------------------------------------------------------
(Updated April 29, 2016, 4:04 p.m.) Review request for geode and anilkumar gingade. Changes ------- implemented review comments. add test for durable client. Bugs: GEODE-1183 https://issues.apache.org/jira/browse/GEODE-1183 Repository: geode Description ------- Current API allows us to create 2 cache servers on the same JVM, then the client will try to create 2 queues to that JVM, one secondary and one primary. But the proxy is actually the same (since there's only one client), so the CCN keeps destroying and recreating the proxy. To fix, we will keep the first proxy and reject the duplicate creating. Then the secondary proxy will automatically become primary. Diffs (updated) ----- geode-core/src/main/java/com/gemstone/gemfire/internal/cache/tier/sockets/CacheClientNotifier.java 1ba2294 geode-wan/src/test/java/com/gemstone/gemfire/internal/cache/wan/CacheClientNotifierDUnitTest.java 8bf819c geode-wan/src/test/java/com/gemstone/gemfire/internal/cache/wan/Simple2CacheServerDUnitTest.java PRE-CREATION Diff: https://reviews.apache.org/r/46625/diff/ Testing ------- Thanks, xiaojian zhou