----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58544/ -----------------------------------------------------------
Review request for geode, Jason Huynh, Jinmei Liao, Jared Stewart, Ken Howe, and Dan Smith. Bugs: GEODE-2632 https://issues.apache.org/jira/browse/GEODE-2632 Repository: geode Description ------- GEODE-2632 refactoring part 5 GEODE-2632: refactor code to use InternalCache instead of GemFireCacheImpl * minor cleanup: delete dead-code and useless javadocs/comments, reduce scope of vars/methods/constants, misc IDE warnings Diffs ----- geode-core/src/main/java/org/apache/geode/cache/query/internal/cq/CqServiceProvider.java cded9c319e9731f74fb6ec14a7f4a02187ae1603 geode-core/src/main/java/org/apache/geode/cache/query/internal/cq/spi/CqServiceFactory.java 68ebbd54d4d2459ec8df27b15ace8c83942723f6 geode-cq/src/main/java/org/apache/geode/cache/query/internal/cq/ClientCQImpl.java 00a0aa5a9e7d8f770f2253e2c48dc6319d6864e3 geode-cq/src/main/java/org/apache/geode/cache/query/internal/cq/CqQueryImpl.java 22b4137c0d52bd84749b9617163b8723d1249ab6 geode-cq/src/main/java/org/apache/geode/cache/query/internal/cq/CqServiceFactoryImpl.java db90632437c80879b3ab213bbad34018809f1c31 geode-cq/src/main/java/org/apache/geode/cache/query/internal/cq/CqServiceImpl.java f1ca8328ad9b9c67e70292093f117cc885d911b5 geode-cq/src/main/java/org/apache/geode/cache/query/internal/cq/CqServiceStatisticsImpl.java ba71143cb271bd177b21e22e00d22a34f69c2834 geode-cq/src/main/java/org/apache/geode/cache/query/internal/cq/CqServiceVsdStats.java 8435c4cea87401a6d29afc65d367d774dc3f4bf6 geode-cq/src/main/java/org/apache/geode/internal/cache/tier/sockets/command/ExecuteCQ.java bcf98063c8ccf7c3d8ba44c9868b8921bb49a64b geode-cq/src/main/java/org/apache/geode/internal/cache/tier/sockets/command/ExecuteCQ61.java f333b4bdafdadad53060fe11ae9d98cc4f4593d4 geode-cq/src/main/java/org/apache/geode/internal/cache/tier/sockets/command/GetDurableCQs.java eac9ed3ad2da2afa92e1230d17b550301631307e geode-cq/src/test/java/org/apache/geode/cache/query/cq/dunit/CqStatsDUnitTest.java 7ace0e8e54bedf94622af5f011627a97c6316dd1 geode-cq/src/test/java/org/apache/geode/cache/query/cq/dunit/CqStatsUsingPoolDUnitTest.java d6068f1ecc88c076c3d1a8dbd3e3bc944766fbaa geode-lucene/src/main/java/org/apache/geode/cache/lucene/internal/distributed/TopEntriesFunctionCollector.java 66c4c0a298156926793f66eee1ac3efc85b27a14 Diff: https://reviews.apache.org/r/58544/diff/1/ Testing ------- precheckin in progress Thanks, Kirk Lund