-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61480/
-----------------------------------------------------------

(Updated Aug. 8, 2017, 9:10 p.m.)


Review request for geode, Emily Yeh, Jared Stewart, Ken Howe, Kirk Lund, and 
Patrick Rhomberg.


Changes
-------

added more tests


Repository: geode


Description
-------

* cq.execute() and cq.executeWithInitialResult() all would still require 
DATA:READ because it will send the result back to the client either initially 
or later.


Diffs (updated)
-----

  geode-core/src/test/java/org/apache/geode/security/SecurityTestUtil.java 
5d5c2148e2eba8054df305942e04f43ea69c0a79 
  
geode-core/src/test/java/org/apache/geode/test/dunit/rules/LocatorServerStartupRule.java
 a455aff315cd26abd398630ff63d8b54b0d1d12b 
  
geode-cq/src/main/java/org/apache/geode/internal/cache/tier/sockets/command/CloseCQ.java
 6748f7d3193babfa668a7ff2846f974f0cdc1cbd 
  
geode-cq/src/main/java/org/apache/geode/internal/cache/tier/sockets/command/ExecuteCQ61.java
 77a608c7e65a2030c0037e9f327cf8c17e9313db 
  
geode-cq/src/main/java/org/apache/geode/internal/cache/tier/sockets/command/StopCQ.java
 a3d51edcc141391e9d818fc0ed7e514d3cb5d6d0 
  
geode-cq/src/test/java/org/apache/geode/internal/cache/tier/sockets/command/CloseCQTest.java
 PRE-CREATION 
  
geode-cq/src/test/java/org/apache/geode/internal/cache/tier/sockets/command/ExecuteCQ61Test.java
 PRE-CREATION 
  
geode-cq/src/test/java/org/apache/geode/internal/cache/tier/sockets/command/StopCQTest.java
 PRE-CREATION 
  geode-cq/src/test/java/org/apache/geode/security/CQClientAuthDUnitTest.java 
89167258ddbc06315068c849255a8530faefe060 
  
geode-cq/src/test/java/org/apache/geode/security/CQPostProcessorDunitTest.java 
45f45abd17cf4f90f96163ebe4f01e67b3b53633 
  geode-cq/src/test/java/org/apache/geode/security/ClientCQAuthDUnitTest.java 
PRE-CREATION 
  
geode-cq/src/test/java/org/apache/geode/security/ClientQueryAuthDUnitTest.java 
cc5dde409c561522ae3739eeaee892079c509ae8 
  geode-cq/src/test/java/org/apache/geode/test/dunit/rules/CQUnitTestRule.java 
PRE-CREATION 


Diff: https://reviews.apache.org/r/61480/diff/2/

Changes: https://reviews.apache.org/r/61480/diff/1-2/


Testing
-------

precheckin running


Thanks,

Jinmei Liao

Reply via email to