geode-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <>
Subject [jira] [Commented] (GEODE-4059) Stop using magic bytes in protobuf connection establishment
Date Thu, 07 Dec 2017 21:23:02 GMT


ASF GitHub Bot commented on GEODE-4059:

WireBaron commented on a change in pull request #1137: GEODE-4059: Changing protobuf handshake
to not need communication mod…

 File path: geode-core/src/main/java/org/apache/geode/internal/cache/tier/
 @@ -22,6 +22,19 @@
  * query methods must be updated.
 public enum CommunicationMode {
+  /**
+   * The first byte of any locator connection will be the high order byte of its gossip version,
+   * which will always be 0. Communication modes should not collide with this value.
+   */
+  ReservedForGossip((byte) 0, "Locator gossip version"),
+  /**
+   * For the new client-server protocol.
+   *
+   * Protobuf handshake messages are specially constructed so that this value will match
the first
+   * byte sent, allowing clients to start protobuf connections with a handshake instead of
+   * communication mode bytes.
+   */
+  ProtobufClientServerProtocol((byte) 10, "Protobuf client"),
 Review comment:
   Kind of, they'll need to back out any magic byte they're currently sending us and just
start each connection with a Handshake message.

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:

> Stop using magic bytes in protobuf connection establishment
> -----------------------------------------------------------
>                 Key: GEODE-4059
>                 URL:
>             Project: Geode
>          Issue Type: New Feature
>          Components: client/server
>            Reporter: Brian Rowe
> When connection to the server, the client currently uses some magic bytes to identify
the protocol being used.  This is non-intuitive and problematic for client developers, and
should not be necessary if we make clever use of a magic protobuf connection establishing
message.  This should also serve as our handshake.

This message was sent by Atlassian JIRA

View raw message