geode-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Brian Rowe <br...@pivotal.io>
Subject Review Request 60486: GEODE-3105: Adding handler for GetRegions
Date Tue, 27 Jun 2017 23:31:15 GMT

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

Review request for geode, Alexander Murmann, Bruce Schuchardt, Galen O'Sullivan, Hitesh Khamesra,
and Udo Kohlmeyer.


Bugs: GEODE-3105
    https://issues.apache.org/jira/browse/GEODE-3105


Repository: geode


Description
-------

Added a handler which will catch incoming getRegion requests and will call into the cache's
rootRegion and return the names of the regions it finds.


Diffs
-----

  geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/GenericProtocolServerConnection.java
8edd83c0f 
  geode-protobuf/src/main/java/org/apache/geode/protocol/protobuf/ProtobufOpsProcessor.java
29d33170c 
  geode-protobuf/src/main/java/org/apache/geode/protocol/protobuf/ProtobufStreamProcessor.java
21dbef5bc 
  geode-protobuf/src/main/java/org/apache/geode/protocol/protobuf/operations/GetRegionsRequestOperationHandler.java
PRE-CREATION 
  geode-protobuf/src/main/proto/region_API.proto adeb011c6 
  geode-protobuf/src/test/java/org/apache/geode/protocol/MessageUtil.java 73d0803b6 
  geode-protobuf/src/test/java/org/apache/geode/protocol/RoundTripCacheConnectionJUnitTest.java
1fbe82159 
  geode-protobuf/src/test/java/org/apache/geode/protocol/protobuf/operations/GetRegionRequestOperationHandlerJUnitTest.java
PRE-CREATION 


Diff: https://reviews.apache.org/r/60486/diff/1/


Testing
-------

Added unit test verifying hanlder behavior.
Added integration test verifying module correctness for getRegion.
Running precheckin.


Thanks,

Brian Rowe


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message