Swap to proto3

ejona86 · ejona86 · commit ff2a28535a8f · 2015-02-25T21:50:37.000-08:00
Benchmark remains proto2 as the .proto has defaults that will need to be
adjusted.
diff --git a/compiler/src/test/proto/test.proto b/compiler/src/test/proto/test.proto
@@ -1,5 +1,5 @@
 // A simple service definition for testing the protoc plugin.
-syntax = "proto2";
+syntax = "proto3";
 
 package grpc.testing;
 
diff --git a/integration-testing/src/main/java/io/grpc/testing/integration/TestServiceImpl.java b/integration-testing/src/main/java/io/grpc/testing/integration/TestServiceImpl.java
@@ -129,10 +129,7 @@ public StreamObserver<Messages.StreamingInputCallRequest> streamingInputCall(
 
       @Override
       public void onValue(StreamingInputCallRequest message) {
-        Payload payload = message.getPayload();
-        if (payload.hasBody()) {
-          totalPayloadSize += payload.getBody().size();
-        }
+        totalPayloadSize += message.getPayload().getBody().size();
       }
 
       @Override
@@ -320,8 +317,7 @@ public Queue<Chunk> toChunkQueue(StreamingOutputCallRequest request) {
     int offset = 0;
     boolean compressable = compressableResponse(request.getResponseType());
     for (ResponseParameters params : request.getResponseParametersList()) {
-      int interval = params.hasIntervalUs() ? params.getIntervalUs() : 0;
-      chunkQueue.add(new Chunk(interval, offset, params.getSize(), compressable));
+      chunkQueue.add(new Chunk(params.getIntervalUs(), offset, params.getSize(), compressable));
 
       // Increment the offset past this chunk.
       // Both buffers need to be circular.
diff --git a/integration-testing/src/main/proto/io/grpc/testing/integration/message_set.proto b/integration-testing/src/main/proto/io/grpc/testing/integration/message_set.proto
diff --git a/integration-testing/src/main/proto/io/grpc/testing/integration/messages.proto b/integration-testing/src/main/proto/io/grpc/testing/integration/messages.proto
@@ -4,12 +4,10 @@
 // Use of this source code is governed by a BSD-style
 // license that can be found in the LICENSE file or at
 // https://developers.google.com/open-source/licenses/bsd
-syntax = "proto2";
+syntax = "proto3";
 
 package stubby.testing;
 
-import "io/grpc/testing/integration/message_set.proto";
-
 option java_package = "io.grpc.testing.integration";
 
 // The type of payload that should be returned.
@@ -56,10 +54,6 @@ message SimpleResponse {
 }
 
 message SimpleContext {
-  // Register MessageSet extension.
-  extend proto2.bridge.MessageSet {
-    optional SimpleContext message_set_extension = 71139615;
-  }
   optional string value = 1;
 }
 
