Re: Review Request 60217: GEODE-2995: adding in protocol handler for protobuf messages

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60217/#review178541 --- Ship it! Ship It! - Galen O'Sullivan On June 21, 2017, 7:01

Re: Review Request 60217: GEODE-2995: adding in protocol handler for protobuf messages

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60217/#review178527 --- Ship it! Ship It! - Hitesh Khamesra On June 21, 2017, 7:01 p

Re: Review Request 60217: GEODE-2995: adding in protocol handler for protobuf messages

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60217/ --- (Updated June 21, 2017, 7:01 p.m.) Review request for geode, Alexander Murmann,

Re: Review Request 60217: GEODE-2995: adding in protocol handler for protobuf messages

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60217/#review178513 --- geode-protobuf/src/main/java/org/apache/geode/protocol/protobuf/E

Re: Review Request 60217: GEODE-2995: adding in protocol handler for protobuf messages

> On June 21, 2017, 12:52 a.m., Udo Kohlmeyer wrote: > > geode-protobuf/src/main/java/org/apache/geode/protocol/operations/registry/OperationsHandlerRegistry.java > > Lines 24 (patched) > > > > > > It is hard to say i

Re: Review Request 60217: GEODE-2995: adding in protocol handler for protobuf messages

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60217/#review178442 --- geode-protobuf/src/main/java/org/apache/geode/ProtobufUtilities.j

Re: Review Request 60217: GEODE-2995: adding in protocol handler for protobuf messages

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60217/#review178441 --- geode-protobuf/build.gradle Lines 26 (patched)

Re: Review Request 60217: GEODE-2995: adding in protocol handler for protobuf messages

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60217/ --- (Updated June 21, 2017, 12:02 a.m.) Review request for geode, Alexander Murmann

Re: Review Request 60217: GEODE-2995: adding in protocol handler for protobuf messages

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60217/#review178412 --- geode-protobuf/src/main/java/org/apache/geode/protocol/OpsProcess

Re: Review Request 60217: GEODE-2995: adding in protocol handler for protobuf messages

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60217/#review178410 --- geode-protobuf/src/main/java/org/apache/geode/ProtobufUtilities.j

Re: Review Request 60217: GEODE-2995: adding in protocol handler for protobuf messages

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60217/#review178419 --- geode-protobuf/src/test/java/org/apache/geode/protocol/handler/Pr

Re: Review Request 60217: GEODE-2995: adding in protocol handler for protobuf messages

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60217/#review178404 --- Good work! I would love to see more doc comments in general. It c

Review Request 60217: GEODE-2995: adding in protocol handler for protobuf messages

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60217/ --- Review request for geode, Alexander Murmann, Bruce Schuchardt, Galen O'Sullivan,