-----------------------------------------------------------
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

Reply via email to