You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@geode.apache.org by bs...@apache.org on 2017/10/05 21:09:36 UTC

[geode] branch develop updated (2502ae0 -> a30de87)

This is an automated email from the ASF dual-hosted git repository.

bschuchardt pushed a change to branch develop
in repository https://gitbox.apache.org/repos/asf/geode.git.


    from 2502ae0  GEODE-3687: GatewayReceiverImpl constructor and create method updates (#859)
     add e2802d2  GEODE-3695 Remove unused protobuf messages from new protocol
     new 2c04972  GEODE-3695 Remove unused protobuf messages from new protocol
     new a30de87  Merge branch 'feature/GEODE_3695' into develop

The 2 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 .../cache/tier/sockets/MessageExecutionContext.java      |  1 -
 geode-protobuf/src/main/proto/clientProtocol.proto       | 10 +---------
 geode-protobuf/src/main/proto/region_API.proto           | 16 ----------------
 geode-protobuf/src/main/proto/server_API.proto           | 11 +----------
 4 files changed, 2 insertions(+), 36 deletions(-)

-- 
To stop receiving notification emails like this one, please contact
['"commits@geode.apache.org" <co...@geode.apache.org>'].

[geode] 02/02: Merge branch 'feature/GEODE_3695' into develop

Posted by bs...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

bschuchardt pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/geode.git

commit a30de87fbe7c539afeb1be3c0339fa5ca3b1c9dd
Merge: 2502ae0 2c04972
Author: Bruce Schuchardt <bs...@pivotal.io>
AuthorDate: Thu Oct 5 13:52:07 2017 -0700

    Merge branch 'feature/GEODE_3695' into develop

 .../cache/tier/sockets/MessageExecutionContext.java      |  1 -
 geode-protobuf/src/main/proto/clientProtocol.proto       | 10 +---------
 geode-protobuf/src/main/proto/region_API.proto           | 16 ----------------
 geode-protobuf/src/main/proto/server_API.proto           | 11 +----------
 4 files changed, 2 insertions(+), 36 deletions(-)

diff --cc geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/MessageExecutionContext.java
index bedb2c7,38f164a..fd1ed4d
--- a/geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/MessageExecutionContext.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/MessageExecutionContext.java
@@@ -85,7 -85,7 +85,6 @@@ public class MessageExecutionContext 
    /**
     * Returns the statistics for recording operation stats. In a unit test environment this may not
     * be a protocol-specific statistics implementation.
-    *
 -   * 
     */
    public ClientProtocolStatistics getStatistics() {
      return statistics;

-- 
To stop receiving notification emails like this one, please contact
"commits@geode.apache.org" <co...@geode.apache.org>.

[geode] 01/02: GEODE-3695 Remove unused protobuf messages from new protocol

Posted by bs...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

bschuchardt pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/geode.git

commit 2c04972162b96aed07b3ff76144b2b22ed6582a1
Author: Bruce Schuchardt <bs...@pivotal.io>
AuthorDate: Thu Oct 5 13:49:21 2017 -0700

    GEODE-3695 Remove unused protobuf messages from new protocol
    
    removed ping messages
---
 .../internal/cache/tier/sockets/MessageExecutionContext.java  |  1 -
 .../geode/protocol/protobuf/utilities/ProtobufUtilities.java  |  2 +-
 geode-protobuf/src/main/proto/clientProtocol.proto            |  2 --
 geode-protobuf/src/main/proto/server_API.proto                | 11 +----------
 4 files changed, 2 insertions(+), 14 deletions(-)

diff --git a/geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/MessageExecutionContext.java b/geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/MessageExecutionContext.java
index d1fc461..38f164a 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/MessageExecutionContext.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/MessageExecutionContext.java
@@ -86,7 +86,6 @@ public class MessageExecutionContext {
    * Returns the statistics for recording operation stats. In a unit test environment this may not
    * be a protocol-specific statistics implementation.
    * 
-   * @return
    */
   public ClientProtocolStatistics getStatistics() {
     return statistics;
diff --git a/geode-protobuf/src/main/java/org/apache/geode/protocol/protobuf/utilities/ProtobufUtilities.java b/geode-protobuf/src/main/java/org/apache/geode/protocol/protobuf/utilities/ProtobufUtilities.java
index 4da8723..1bac88f 100644
--- a/geode-protobuf/src/main/java/org/apache/geode/protocol/protobuf/utilities/ProtobufUtilities.java
+++ b/geode-protobuf/src/main/java/org/apache/geode/protocol/protobuf/utilities/ProtobufUtilities.java
@@ -84,7 +84,7 @@ public abstract class ProtobufUtilities {
 
   /**
    * Creates a protobuf (key, value) pair from unencoded data. if the value is null, it will be
-   * unset in the {@link BasicTypes.Entry}.
+   * unset in the BasicTypes.Entry.
    *
    * The key MUST NOT be null.
    *
diff --git a/geode-protobuf/src/main/proto/clientProtocol.proto b/geode-protobuf/src/main/proto/clientProtocol.proto
index 8dc3f57..f91949f 100644
--- a/geode-protobuf/src/main/proto/clientProtocol.proto
+++ b/geode-protobuf/src/main/proto/clientProtocol.proto
@@ -49,7 +49,6 @@ message Request {
         RemoveRequest removeRequest = 6;
         RemoveAllRequest removeAllRequest = 7;
 
-//        PingRequest pingRequest = 41;
         GetAvailableServersRequest getAvailableServersRequest = 42;
         GetRegionNamesRequest getRegionNamesRequest = 43;
         GetRegionRequest getRegionRequest = 44;
@@ -67,7 +66,6 @@ message Response {
 
         ErrorResponse errorResponse = 13;
 
-//        PingResponse pingResponse = 41;
         GetAvailableServersResponse getAvailableServersResponse = 42;
         GetRegionNamesResponse getRegionNamesResponse = 43;
         GetRegionResponse getRegionResponse = 44;
diff --git a/geode-protobuf/src/main/proto/server_API.proto b/geode-protobuf/src/main/proto/server_API.proto
index ae608da..0ea77a8 100644
--- a/geode-protobuf/src/main/proto/server_API.proto
+++ b/geode-protobuf/src/main/proto/server_API.proto
@@ -23,19 +23,10 @@ package org.apache.geode.internal.protocol.protobuf;
 
 import "basicTypes.proto";
 
-// PingResponse sends the same number back.
-message PingRequest {
-    int32 sequenceNumber = 1;
-}
-
-message PingResponse {
-    int32 sequenceNumber = 1;
-}
-
 message GetAvailableServersRequest {
 
 }
 
 message GetAvailableServersResponse {
     repeated Server servers = 1;
-}
\ No newline at end of file
+}

-- 
To stop receiving notification emails like this one, please contact
"commits@geode.apache.org" <co...@geode.apache.org>.