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