diff options
author | Carl Mastrangelo <notcarl@google.com> | 2018-06-05 11:19:28 -0700 |
---|---|---|
committer | GitHub <noreply@github.com> | 2018-06-05 11:19:28 -0700 |
commit | 4c4fda3e5d4a08972d48f5e1435bdc5f335350ee (patch) | |
tree | 81980fa1f66271db351a33fe6594e21123aa286c /compiler | |
parent | 8fd762fb8e6d11de6c09c84967620bdb84b71822 (diff) | |
download | grpc-grpc-java-4c4fda3e5d4a08972d48f5e1435bdc5f335350ee.tar.gz |
stub: remove static Method descriptors and stabilize method accessors
Diffstat (limited to 'compiler')
-rw-r--r-- | compiler/src/java_plugin/cpp/java_generator.cpp | 55 | ||||
-rw-r--r-- | compiler/src/test/golden/TestService.java.txt | 115 | ||||
-rw-r--r-- | compiler/src/testLite/golden/TestService.java.txt | 115 | ||||
-rw-r--r-- | compiler/src/testNano/golden/TestService.java.txt | 125 |
4 files changed, 100 insertions, 310 deletions
diff --git a/compiler/src/java_plugin/cpp/java_generator.cpp b/compiler/src/java_plugin/cpp/java_generator.cpp index 411d6738a..e3d39455f 100644 --- a/compiler/src/java_plugin/cpp/java_generator.cpp +++ b/compiler/src/java_plugin/cpp/java_generator.cpp @@ -140,10 +140,6 @@ static inline string MethodPropertiesGetterName(const MethodDescriptor* method) return MixedLower("get_" + method->name() + "_method"); } -static inline string MethodPropertiesGetterHelperName(const MethodDescriptor* method) { - return MixedLower("get_" + method->name() + "_method_helper"); -} - static inline string MethodIdFieldName(const MethodDescriptor* method) { return "METHODID_" + ToAllUpperCase(method->name()); } @@ -385,7 +381,6 @@ static void PrintMethodFields( (*vars)["method_field_name"] = MethodPropertiesFieldName(method); (*vars)["method_new_field_name"] = MethodPropertiesGetterName(method); (*vars)["method_method_name"] = MethodPropertiesGetterName(method); - (*vars)["method_method_name_helper"] = MethodPropertiesGetterHelperName(method); bool client_streaming = method->client_streaming(); bool server_streaming = method->server_streaming(); if (client_streaming) { @@ -410,22 +405,11 @@ static void PrintMethodFields( *vars, "private static final int ARG_IN_$method_field_name$ = $arg_in_id$;\n" "private static final int ARG_OUT_$method_field_name$ = $arg_out_id$;\n" - "@$ExperimentalApi$(\"https://github.com/grpc/grpc-java/issues/1901\")\n" - "@$Deprecated$ // Use {@link #$method_method_name$()} instead. \n" - "public static final $MethodDescriptor$<$input_type$,\n" - " $output_type$> $method_field_name$ = $method_method_name_helper$();\n" - "\n" "private static volatile $MethodDescriptor$<$input_type$,\n" " $output_type$> $method_new_field_name$;\n" "\n" - "@$ExperimentalApi$(\"https://github.com/grpc/grpc-java/issues/1901\")\n" "public static $MethodDescriptor$<$input_type$,\n" " $output_type$> $method_method_name$() {\n" - " return $method_method_name_helper$();\n" - "}\n" - "\n" - "private static $MethodDescriptor$<$input_type$,\n" - " $output_type$> $method_method_name_helper$() {\n" " $MethodDescriptor$<$input_type$, $output_type$> $method_new_field_name$;\n" " if (($method_new_field_name$ = $service_class_name$.$method_new_field_name$) == null) {\n" " synchronized ($service_class_name$.class) {\n" @@ -445,7 +429,8 @@ static void PrintMethodFields( " }\n" " }\n" " return $method_new_field_name$;\n" - "}\n"); + "}\n" + "\n"); } else { if (flavor == ProtoFlavor::LITE) { (*vars)["ProtoUtils"] = "io.grpc.protobuf.lite.ProtoLiteUtils"; @@ -454,22 +439,11 @@ static void PrintMethodFields( } p->Print( *vars, - "@$ExperimentalApi$(\"https://github.com/grpc/grpc-java/issues/1901\")\n" - "@$Deprecated$ // Use {@link #$method_method_name$()} instead. \n" - "public static final $MethodDescriptor$<$input_type$,\n" - " $output_type$> $method_field_name$ = $method_method_name_helper$();\n" - "\n" "private static volatile $MethodDescriptor$<$input_type$,\n" " $output_type$> $method_new_field_name$;\n" "\n" - "@$ExperimentalApi$(\"https://github.com/grpc/grpc-java/issues/1901\")\n" "public static $MethodDescriptor$<$input_type$,\n" " $output_type$> $method_method_name$() {\n" - " return $method_method_name_helper$();\n" - "}\n" - "\n" - "private static $MethodDescriptor$<$input_type$,\n" - " $output_type$> $method_method_name_helper$() {\n" " $MethodDescriptor$<$input_type$, $output_type$> $method_new_field_name$;\n" " if (($method_new_field_name$ = $service_class_name$.$method_new_field_name$) == null) {\n" " synchronized ($service_class_name$.class) {\n" @@ -500,11 +474,11 @@ static void PrintMethodFields( " }\n" " }\n" " return $method_new_field_name$;\n" - "}\n"); + "}\n" + "\n"); } } - p->Print("\n"); if (flavor == ProtoFlavor::NANO) { p->Print( @@ -674,7 +648,7 @@ static void PrintStub( (*vars)["output_type"] = MessageFullJavaName(generate_nano, method->output_type()); (*vars)["lower_method_name"] = LowerMethodName(method); - (*vars)["method_method_name_helper"] = MethodPropertiesGetterHelperName(method); + (*vars)["method_method_name"] = MethodPropertiesGetterName(method); bool client_streaming = method->client_streaming(); bool server_streaming = method->server_streaming(); @@ -754,11 +728,11 @@ static void PrintStub( if (client_streaming) { p->Print( *vars, - "return asyncUnimplementedStreamingCall($method_method_name_helper$(), responseObserver);\n"); + "return asyncUnimplementedStreamingCall($method_method_name$(), responseObserver);\n"); } else { p->Print( *vars, - "asyncUnimplementedUnaryCall($method_method_name_helper$(), responseObserver);\n"); + "asyncUnimplementedUnaryCall($method_method_name$(), responseObserver);\n"); } break; default: @@ -779,7 +753,7 @@ static void PrintStub( p->Print( *vars, "return $calls_method$(\n" - " getChannel(), $method_method_name_helper$(), getCallOptions(), $params$);\n"); + " getChannel(), $method_method_name$(), getCallOptions(), $params$);\n"); break; case ASYNC_CALL: if (server_streaming) { @@ -803,7 +777,7 @@ static void PrintStub( p->Print( *vars, "$last_line_prefix$$calls_method$(\n" - " getChannel().newCall($method_method_name_helper$(), getCallOptions()), $params$);\n"); + " getChannel().newCall($method_method_name$(), getCallOptions()), $params$);\n"); break; case FUTURE_CALL: GRPC_CODEGEN_CHECK(!client_streaming && !server_streaming) @@ -814,7 +788,7 @@ static void PrintStub( p->Print( *vars, "return $calls_method$(\n" - " getChannel().newCall($method_method_name_helper$(), getCallOptions()), request);\n"); + " getChannel().newCall($method_method_name$(), getCallOptions()), request);\n"); break; } } @@ -1038,8 +1012,8 @@ static void PrintGetServiceDescriptorMethod(const ServiceDescriptor* service, } for (int i = 0; i < service->method_count(); ++i) { const MethodDescriptor* method = service->method(i); - (*vars)["method_method_name_helper"] = MethodPropertiesGetterHelperName(method); - p->Print(*vars, "\n.addMethod($method_method_name_helper$())"); + (*vars)["method_method_name"] = MethodPropertiesGetterName(method); + p->Print(*vars, "\n.addMethod($method_method_name$())"); } p->Print("\n.build();\n"); p->Outdent(); @@ -1070,7 +1044,7 @@ static void PrintBindServiceMethodBody(const ServiceDescriptor* service, for (int i = 0; i < service->method_count(); ++i) { const MethodDescriptor* method = service->method(i); (*vars)["lower_method_name"] = LowerMethodName(method); - (*vars)["method_method_name_helper"] = MethodPropertiesGetterHelperName(method); + (*vars)["method_method_name"] = MethodPropertiesGetterName(method); (*vars)["input_type"] = MessageFullJavaName(generate_nano, method->input_type()); (*vars)["output_type"] = MessageFullJavaName(generate_nano, @@ -1095,7 +1069,7 @@ static void PrintBindServiceMethodBody(const ServiceDescriptor* service, p->Indent(); p->Print( *vars, - "$method_method_name_helper$(),\n" + "$method_method_name$(),\n" "$calls_method$(\n"); p->Indent(); p->Print( @@ -1268,7 +1242,6 @@ void GenerateService(const ServiceDescriptor* service, vars["Generated"] = "javax.annotation.Generated"; vars["ListenableFuture"] = "com.google.common.util.concurrent.ListenableFuture"; - vars["ExperimentalApi"] = "io.grpc.ExperimentalApi"; Printer printer(out, '$'); string package_name = ServiceJavaPackage(service->file(), diff --git a/compiler/src/test/golden/TestService.java.txt b/compiler/src/test/golden/TestService.java.txt index 55d5fab2f..401ea4193 100644 --- a/compiler/src/test/golden/TestService.java.txt +++ b/compiler/src/test/golden/TestService.java.txt @@ -30,22 +30,11 @@ public final class TestServiceGrpc { public static final String SERVICE_NAME = "grpc.testing.compiler.TestService"; // Static method descriptors that strictly reflect the proto. - @io.grpc.ExperimentalApi("https://github.com/grpc/grpc-java/issues/1901") - @java.lang.Deprecated // Use {@link #getUnaryCallMethod()} instead. - public static final io.grpc.MethodDescriptor<io.grpc.testing.compiler.Test.SimpleRequest, - io.grpc.testing.compiler.Test.SimpleResponse> METHOD_UNARY_CALL = getUnaryCallMethodHelper(); - private static volatile io.grpc.MethodDescriptor<io.grpc.testing.compiler.Test.SimpleRequest, io.grpc.testing.compiler.Test.SimpleResponse> getUnaryCallMethod; - @io.grpc.ExperimentalApi("https://github.com/grpc/grpc-java/issues/1901") public static io.grpc.MethodDescriptor<io.grpc.testing.compiler.Test.SimpleRequest, io.grpc.testing.compiler.Test.SimpleResponse> getUnaryCallMethod() { - return getUnaryCallMethodHelper(); - } - - private static io.grpc.MethodDescriptor<io.grpc.testing.compiler.Test.SimpleRequest, - io.grpc.testing.compiler.Test.SimpleResponse> getUnaryCallMethodHelper() { io.grpc.MethodDescriptor<io.grpc.testing.compiler.Test.SimpleRequest, io.grpc.testing.compiler.Test.SimpleResponse> getUnaryCallMethod; if ((getUnaryCallMethod = TestServiceGrpc.getUnaryCallMethod) == null) { synchronized (TestServiceGrpc.class) { @@ -67,22 +56,12 @@ public final class TestServiceGrpc { } return getUnaryCallMethod; } - @io.grpc.ExperimentalApi("https://github.com/grpc/grpc-java/issues/1901") - @java.lang.Deprecated // Use {@link #getStreamingOutputCallMethod()} instead. - public static final io.grpc.MethodDescriptor<io.grpc.testing.compiler.Test.StreamingOutputCallRequest, - io.grpc.testing.compiler.Test.StreamingOutputCallResponse> METHOD_STREAMING_OUTPUT_CALL = getStreamingOutputCallMethodHelper(); private static volatile io.grpc.MethodDescriptor<io.grpc.testing.compiler.Test.StreamingOutputCallRequest, io.grpc.testing.compiler.Test.StreamingOutputCallResponse> getStreamingOutputCallMethod; - @io.grpc.ExperimentalApi("https://github.com/grpc/grpc-java/issues/1901") public static io.grpc.MethodDescriptor<io.grpc.testing.compiler.Test.StreamingOutputCallRequest, io.grpc.testing.compiler.Test.StreamingOutputCallResponse> getStreamingOutputCallMethod() { - return getStreamingOutputCallMethodHelper(); - } - - private static io.grpc.MethodDescriptor<io.grpc.testing.compiler.Test.StreamingOutputCallRequest, - io.grpc.testing.compiler.Test.StreamingOutputCallResponse> getStreamingOutputCallMethodHelper() { io.grpc.MethodDescriptor<io.grpc.testing.compiler.Test.StreamingOutputCallRequest, io.grpc.testing.compiler.Test.StreamingOutputCallResponse> getStreamingOutputCallMethod; if ((getStreamingOutputCallMethod = TestServiceGrpc.getStreamingOutputCallMethod) == null) { synchronized (TestServiceGrpc.class) { @@ -104,22 +83,12 @@ public final class TestServiceGrpc { } return getStreamingOutputCallMethod; } - @io.grpc.ExperimentalApi("https://github.com/grpc/grpc-java/issues/1901") - @java.lang.Deprecated // Use {@link #getStreamingInputCallMethod()} instead. - public static final io.grpc.MethodDescriptor<io.grpc.testing.compiler.Test.StreamingInputCallRequest, - io.grpc.testing.compiler.Test.StreamingInputCallResponse> METHOD_STREAMING_INPUT_CALL = getStreamingInputCallMethodHelper(); private static volatile io.grpc.MethodDescriptor<io.grpc.testing.compiler.Test.StreamingInputCallRequest, io.grpc.testing.compiler.Test.StreamingInputCallResponse> getStreamingInputCallMethod; - @io.grpc.ExperimentalApi("https://github.com/grpc/grpc-java/issues/1901") public static io.grpc.MethodDescriptor<io.grpc.testing.compiler.Test.StreamingInputCallRequest, io.grpc.testing.compiler.Test.StreamingInputCallResponse> getStreamingInputCallMethod() { - return getStreamingInputCallMethodHelper(); - } - - private static io.grpc.MethodDescriptor<io.grpc.testing.compiler.Test.StreamingInputCallRequest, - io.grpc.testing.compiler.Test.StreamingInputCallResponse> getStreamingInputCallMethodHelper() { io.grpc.MethodDescriptor<io.grpc.testing.compiler.Test.StreamingInputCallRequest, io.grpc.testing.compiler.Test.StreamingInputCallResponse> getStreamingInputCallMethod; if ((getStreamingInputCallMethod = TestServiceGrpc.getStreamingInputCallMethod) == null) { synchronized (TestServiceGrpc.class) { @@ -141,22 +110,12 @@ public final class TestServiceGrpc { } return getStreamingInputCallMethod; } - @io.grpc.ExperimentalApi("https://github.com/grpc/grpc-java/issues/1901") - @java.lang.Deprecated // Use {@link #getFullBidiCallMethod()} instead. - public static final io.grpc.MethodDescriptor<io.grpc.testing.compiler.Test.StreamingOutputCallRequest, - io.grpc.testing.compiler.Test.StreamingOutputCallResponse> METHOD_FULL_BIDI_CALL = getFullBidiCallMethodHelper(); private static volatile io.grpc.MethodDescriptor<io.grpc.testing.compiler.Test.StreamingOutputCallRequest, io.grpc.testing.compiler.Test.StreamingOutputCallResponse> getFullBidiCallMethod; - @io.grpc.ExperimentalApi("https://github.com/grpc/grpc-java/issues/1901") public static io.grpc.MethodDescriptor<io.grpc.testing.compiler.Test.StreamingOutputCallRequest, io.grpc.testing.compiler.Test.StreamingOutputCallResponse> getFullBidiCallMethod() { - return getFullBidiCallMethodHelper(); - } - - private static io.grpc.MethodDescriptor<io.grpc.testing.compiler.Test.StreamingOutputCallRequest, - io.grpc.testing.compiler.Test.StreamingOutputCallResponse> getFullBidiCallMethodHelper() { io.grpc.MethodDescriptor<io.grpc.testing.compiler.Test.StreamingOutputCallRequest, io.grpc.testing.compiler.Test.StreamingOutputCallResponse> getFullBidiCallMethod; if ((getFullBidiCallMethod = TestServiceGrpc.getFullBidiCallMethod) == null) { synchronized (TestServiceGrpc.class) { @@ -178,22 +137,12 @@ public final class TestServiceGrpc { } return getFullBidiCallMethod; } - @io.grpc.ExperimentalApi("https://github.com/grpc/grpc-java/issues/1901") - @java.lang.Deprecated // Use {@link #getHalfBidiCallMethod()} instead. - public static final io.grpc.MethodDescriptor<io.grpc.testing.compiler.Test.StreamingOutputCallRequest, - io.grpc.testing.compiler.Test.StreamingOutputCallResponse> METHOD_HALF_BIDI_CALL = getHalfBidiCallMethodHelper(); private static volatile io.grpc.MethodDescriptor<io.grpc.testing.compiler.Test.StreamingOutputCallRequest, io.grpc.testing.compiler.Test.StreamingOutputCallResponse> getHalfBidiCallMethod; - @io.grpc.ExperimentalApi("https://github.com/grpc/grpc-java/issues/1901") public static io.grpc.MethodDescriptor<io.grpc.testing.compiler.Test.StreamingOutputCallRequest, io.grpc.testing.compiler.Test.StreamingOutputCallResponse> getHalfBidiCallMethod() { - return getHalfBidiCallMethodHelper(); - } - - private static io.grpc.MethodDescriptor<io.grpc.testing.compiler.Test.StreamingOutputCallRequest, - io.grpc.testing.compiler.Test.StreamingOutputCallResponse> getHalfBidiCallMethodHelper() { io.grpc.MethodDescriptor<io.grpc.testing.compiler.Test.StreamingOutputCallRequest, io.grpc.testing.compiler.Test.StreamingOutputCallResponse> getHalfBidiCallMethod; if ((getHalfBidiCallMethod = TestServiceGrpc.getHalfBidiCallMethod) == null) { synchronized (TestServiceGrpc.class) { @@ -215,22 +164,12 @@ public final class TestServiceGrpc { } return getHalfBidiCallMethod; } - @io.grpc.ExperimentalApi("https://github.com/grpc/grpc-java/issues/1901") - @java.lang.Deprecated // Use {@link #getImportMethod()} instead. - public static final io.grpc.MethodDescriptor<io.grpc.testing.compiler.Test.StreamingInputCallRequest, - io.grpc.testing.compiler.Test.StreamingInputCallResponse> METHOD_IMPORT = getImportMethodHelper(); private static volatile io.grpc.MethodDescriptor<io.grpc.testing.compiler.Test.StreamingInputCallRequest, io.grpc.testing.compiler.Test.StreamingInputCallResponse> getImportMethod; - @io.grpc.ExperimentalApi("https://github.com/grpc/grpc-java/issues/1901") public static io.grpc.MethodDescriptor<io.grpc.testing.compiler.Test.StreamingInputCallRequest, io.grpc.testing.compiler.Test.StreamingInputCallResponse> getImportMethod() { - return getImportMethodHelper(); - } - - private static io.grpc.MethodDescriptor<io.grpc.testing.compiler.Test.StreamingInputCallRequest, - io.grpc.testing.compiler.Test.StreamingInputCallResponse> getImportMethodHelper() { io.grpc.MethodDescriptor<io.grpc.testing.compiler.Test.StreamingInputCallRequest, io.grpc.testing.compiler.Test.StreamingInputCallResponse> getImportMethod; if ((getImportMethod = TestServiceGrpc.getImportMethod) == null) { synchronized (TestServiceGrpc.class) { @@ -291,7 +230,7 @@ public final class TestServiceGrpc { */ public void unaryCall(io.grpc.testing.compiler.Test.SimpleRequest request, io.grpc.stub.StreamObserver<io.grpc.testing.compiler.Test.SimpleResponse> responseObserver) { - asyncUnimplementedUnaryCall(getUnaryCallMethodHelper(), responseObserver); + asyncUnimplementedUnaryCall(getUnaryCallMethod(), responseObserver); } /** @@ -302,7 +241,7 @@ public final class TestServiceGrpc { */ public void streamingOutputCall(io.grpc.testing.compiler.Test.StreamingOutputCallRequest request, io.grpc.stub.StreamObserver<io.grpc.testing.compiler.Test.StreamingOutputCallResponse> responseObserver) { - asyncUnimplementedUnaryCall(getStreamingOutputCallMethodHelper(), responseObserver); + asyncUnimplementedUnaryCall(getStreamingOutputCallMethod(), responseObserver); } /** @@ -313,7 +252,7 @@ public final class TestServiceGrpc { */ public io.grpc.stub.StreamObserver<io.grpc.testing.compiler.Test.StreamingInputCallRequest> streamingInputCall( io.grpc.stub.StreamObserver<io.grpc.testing.compiler.Test.StreamingInputCallResponse> responseObserver) { - return asyncUnimplementedStreamingCall(getStreamingInputCallMethodHelper(), responseObserver); + return asyncUnimplementedStreamingCall(getStreamingInputCallMethod(), responseObserver); } /** @@ -325,7 +264,7 @@ public final class TestServiceGrpc { */ public io.grpc.stub.StreamObserver<io.grpc.testing.compiler.Test.StreamingOutputCallRequest> fullBidiCall( io.grpc.stub.StreamObserver<io.grpc.testing.compiler.Test.StreamingOutputCallResponse> responseObserver) { - return asyncUnimplementedStreamingCall(getFullBidiCallMethodHelper(), responseObserver); + return asyncUnimplementedStreamingCall(getFullBidiCallMethod(), responseObserver); } /** @@ -338,7 +277,7 @@ public final class TestServiceGrpc { */ public io.grpc.stub.StreamObserver<io.grpc.testing.compiler.Test.StreamingOutputCallRequest> halfBidiCall( io.grpc.stub.StreamObserver<io.grpc.testing.compiler.Test.StreamingOutputCallResponse> responseObserver) { - return asyncUnimplementedStreamingCall(getHalfBidiCallMethodHelper(), responseObserver); + return asyncUnimplementedStreamingCall(getHalfBidiCallMethod(), responseObserver); } /** @@ -349,48 +288,48 @@ public final class TestServiceGrpc { */ public io.grpc.stub.StreamObserver<io.grpc.testing.compiler.Test.StreamingInputCallRequest> import_( io.grpc.stub.StreamObserver<io.grpc.testing.compiler.Test.StreamingInputCallResponse> responseObserver) { - return asyncUnimplementedStreamingCall(getImportMethodHelper(), responseObserver); + return asyncUnimplementedStreamingCall(getImportMethod(), responseObserver); } @java.lang.Override public final io.grpc.ServerServiceDefinition bindService() { return io.grpc.ServerServiceDefinition.builder(getServiceDescriptor()) .addMethod( - getUnaryCallMethodHelper(), + getUnaryCallMethod(), asyncUnaryCall( new MethodHandlers< io.grpc.testing.compiler.Test.SimpleRequest, io.grpc.testing.compiler.Test.SimpleResponse>( this, METHODID_UNARY_CALL))) .addMethod( - getStreamingOutputCallMethodHelper(), + getStreamingOutputCallMethod(), asyncServerStreamingCall( new MethodHandlers< io.grpc.testing.compiler.Test.StreamingOutputCallRequest, io.grpc.testing.compiler.Test.StreamingOutputCallResponse>( this, METHODID_STREAMING_OUTPUT_CALL))) .addMethod( - getStreamingInputCallMethodHelper(), + getStreamingInputCallMethod(), asyncClientStreamingCall( new MethodHandlers< io.grpc.testing.compiler.Test.StreamingInputCallRequest, io.grpc.testing.compiler.Test.StreamingInputCallResponse>( this, METHODID_STREAMING_INPUT_CALL))) .addMethod( - getFullBidiCallMethodHelper(), + getFullBidiCallMethod(), asyncBidiStreamingCall( new MethodHandlers< io.grpc.testing.compiler.Test.StreamingOutputCallRequest, io.grpc.testing.compiler.Test.StreamingOutputCallResponse>( this, METHODID_FULL_BIDI_CALL))) .addMethod( - getHalfBidiCallMethodHelper(), + getHalfBidiCallMethod(), asyncBidiStreamingCall( new MethodHandlers< io.grpc.testing.compiler.Test.StreamingOutputCallRequest, io.grpc.testing.compiler.Test.StreamingOutputCallResponse>( this, METHODID_HALF_BIDI_CALL))) .addMethod( - getImportMethodHelper(), + getImportMethod(), asyncBidiStreamingCall( new MethodHandlers< io.grpc.testing.compiler.Test.StreamingInputCallRequest, @@ -430,7 +369,7 @@ public final class TestServiceGrpc { public void unaryCall(io.grpc.testing.compiler.Test.SimpleRequest request, io.grpc.stub.StreamObserver<io.grpc.testing.compiler.Test.SimpleResponse> responseObserver) { asyncUnaryCall( - getChannel().newCall(getUnaryCallMethodHelper(), getCallOptions()), request, responseObserver); + getChannel().newCall(getUnaryCallMethod(), getCallOptions()), request, responseObserver); } /** @@ -442,7 +381,7 @@ public final class TestServiceGrpc { public void streamingOutputCall(io.grpc.testing.compiler.Test.StreamingOutputCallRequest request, io.grpc.stub.StreamObserver<io.grpc.testing.compiler.Test.StreamingOutputCallResponse> responseObserver) { asyncServerStreamingCall( - getChannel().newCall(getStreamingOutputCallMethodHelper(), getCallOptions()), request, responseObserver); + getChannel().newCall(getStreamingOutputCallMethod(), getCallOptions()), request, responseObserver); } /** @@ -454,7 +393,7 @@ public final class TestServiceGrpc { public io.grpc.stub.StreamObserver<io.grpc.testing.compiler.Test.StreamingInputCallRequest> streamingInputCall( io.grpc.stub.StreamObserver<io.grpc.testing.compiler.Test.StreamingInputCallResponse> responseObserver) { return asyncClientStreamingCall( - getChannel().newCall(getStreamingInputCallMethodHelper(), getCallOptions()), responseObserver); + getChannel().newCall(getStreamingInputCallMethod(), getCallOptions()), responseObserver); } /** @@ -467,7 +406,7 @@ public final class TestServiceGrpc { public io.grpc.stub.StreamObserver<io.grpc.testing.compiler.Test.StreamingOutputCallRequest> fullBidiCall( io.grpc.stub.StreamObserver<io.grpc.testing.compiler.Test.StreamingOutputCallResponse> responseObserver) { return asyncBidiStreamingCall( - getChannel().newCall(getFullBidiCallMethodHelper(), getCallOptions()), responseObserver); + getChannel().newCall(getFullBidiCallMethod(), getCallOptions()), responseObserver); } /** @@ -481,7 +420,7 @@ public final class TestServiceGrpc { public io.grpc.stub.StreamObserver<io.grpc.testing.compiler.Test.StreamingOutputCallRequest> halfBidiCall( io.grpc.stub.StreamObserver<io.grpc.testing.compiler.Test.StreamingOutputCallResponse> responseObserver) { return asyncBidiStreamingCall( - getChannel().newCall(getHalfBidiCallMethodHelper(), getCallOptions()), responseObserver); + getChannel().newCall(getHalfBidiCallMethod(), getCallOptions()), responseObserver); } /** @@ -493,7 +432,7 @@ public final class TestServiceGrpc { public io.grpc.stub.StreamObserver<io.grpc.testing.compiler.Test.StreamingInputCallRequest> import_( io.grpc.stub.StreamObserver<io.grpc.testing.compiler.Test.StreamingInputCallResponse> responseObserver) { return asyncBidiStreamingCall( - getChannel().newCall(getImportMethodHelper(), getCallOptions()), responseObserver); + getChannel().newCall(getImportMethod(), getCallOptions()), responseObserver); } } @@ -526,7 +465,7 @@ public final class TestServiceGrpc { */ public io.grpc.testing.compiler.Test.SimpleResponse unaryCall(io.grpc.testing.compiler.Test.SimpleRequest request) { return blockingUnaryCall( - getChannel(), getUnaryCallMethodHelper(), getCallOptions(), request); + getChannel(), getUnaryCallMethod(), getCallOptions(), request); } /** @@ -538,7 +477,7 @@ public final class TestServiceGrpc { public java.util.Iterator<io.grpc.testing.compiler.Test.StreamingOutputCallResponse> streamingOutputCall( io.grpc.testing.compiler.Test.StreamingOutputCallRequest request) { return blockingServerStreamingCall( - getChannel(), getStreamingOutputCallMethodHelper(), getCallOptions(), request); + getChannel(), getStreamingOutputCallMethod(), getCallOptions(), request); } } @@ -572,7 +511,7 @@ public final class TestServiceGrpc { public com.google.common.util.concurrent.ListenableFuture<io.grpc.testing.compiler.Test.SimpleResponse> unaryCall( io.grpc.testing.compiler.Test.SimpleRequest request) { return futureUnaryCall( - getChannel().newCall(getUnaryCallMethodHelper(), getCallOptions()), request); + getChannel().newCall(getUnaryCallMethod(), getCallOptions()), request); } } @@ -681,12 +620,12 @@ public final class TestServiceGrpc { if (result == null) { serviceDescriptor = result = io.grpc.ServiceDescriptor.newBuilder(SERVICE_NAME) .setSchemaDescriptor(new TestServiceFileDescriptorSupplier()) - .addMethod(getUnaryCallMethodHelper()) - .addMethod(getStreamingOutputCallMethodHelper()) - .addMethod(getStreamingInputCallMethodHelper()) - .addMethod(getFullBidiCallMethodHelper()) - .addMethod(getHalfBidiCallMethodHelper()) - .addMethod(getImportMethodHelper()) + .addMethod(getUnaryCallMethod()) + .addMethod(getStreamingOutputCallMethod()) + .addMethod(getStreamingInputCallMethod()) + .addMethod(getFullBidiCallMethod()) + .addMethod(getHalfBidiCallMethod()) + .addMethod(getImportMethod()) .build(); } } diff --git a/compiler/src/testLite/golden/TestService.java.txt b/compiler/src/testLite/golden/TestService.java.txt index 4b0f2ae14..c00145b0d 100644 --- a/compiler/src/testLite/golden/TestService.java.txt +++ b/compiler/src/testLite/golden/TestService.java.txt @@ -30,22 +30,11 @@ public final class TestServiceGrpc { public static final String SERVICE_NAME = "grpc.testing.compiler.TestService"; // Static method descriptors that strictly reflect the proto. - @io.grpc.ExperimentalApi("https://github.com/grpc/grpc-java/issues/1901") - @java.lang.Deprecated // Use {@link #getUnaryCallMethod()} instead. - public static final io.grpc.MethodDescriptor<io.grpc.testing.compiler.Test.SimpleRequest, - io.grpc.testing.compiler.Test.SimpleResponse> METHOD_UNARY_CALL = getUnaryCallMethodHelper(); - private static volatile io.grpc.MethodDescriptor<io.grpc.testing.compiler.Test.SimpleRequest, io.grpc.testing.compiler.Test.SimpleResponse> getUnaryCallMethod; - @io.grpc.ExperimentalApi("https://github.com/grpc/grpc-java/issues/1901") public static io.grpc.MethodDescriptor<io.grpc.testing.compiler.Test.SimpleRequest, io.grpc.testing.compiler.Test.SimpleResponse> getUnaryCallMethod() { - return getUnaryCallMethodHelper(); - } - - private static io.grpc.MethodDescriptor<io.grpc.testing.compiler.Test.SimpleRequest, - io.grpc.testing.compiler.Test.SimpleResponse> getUnaryCallMethodHelper() { io.grpc.MethodDescriptor<io.grpc.testing.compiler.Test.SimpleRequest, io.grpc.testing.compiler.Test.SimpleResponse> getUnaryCallMethod; if ((getUnaryCallMethod = TestServiceGrpc.getUnaryCallMethod) == null) { synchronized (TestServiceGrpc.class) { @@ -66,22 +55,12 @@ public final class TestServiceGrpc { } return getUnaryCallMethod; } - @io.grpc.ExperimentalApi("https://github.com/grpc/grpc-java/issues/1901") - @java.lang.Deprecated // Use {@link #getStreamingOutputCallMethod()} instead. - public static final io.grpc.MethodDescriptor<io.grpc.testing.compiler.Test.StreamingOutputCallRequest, - io.grpc.testing.compiler.Test.StreamingOutputCallResponse> METHOD_STREAMING_OUTPUT_CALL = getStreamingOutputCallMethodHelper(); private static volatile io.grpc.MethodDescriptor<io.grpc.testing.compiler.Test.StreamingOutputCallRequest, io.grpc.testing.compiler.Test.StreamingOutputCallResponse> getStreamingOutputCallMethod; - @io.grpc.ExperimentalApi("https://github.com/grpc/grpc-java/issues/1901") public static io.grpc.MethodDescriptor<io.grpc.testing.compiler.Test.StreamingOutputCallRequest, io.grpc.testing.compiler.Test.StreamingOutputCallResponse> getStreamingOutputCallMethod() { - return getStreamingOutputCallMethodHelper(); - } - - private static io.grpc.MethodDescriptor<io.grpc.testing.compiler.Test.StreamingOutputCallRequest, - io.grpc.testing.compiler.Test.StreamingOutputCallResponse> getStreamingOutputCallMethodHelper() { io.grpc.MethodDescriptor<io.grpc.testing.compiler.Test.StreamingOutputCallRequest, io.grpc.testing.compiler.Test.StreamingOutputCallResponse> getStreamingOutputCallMethod; if ((getStreamingOutputCallMethod = TestServiceGrpc.getStreamingOutputCallMethod) == null) { synchronized (TestServiceGrpc.class) { @@ -102,22 +81,12 @@ public final class TestServiceGrpc { } return getStreamingOutputCallMethod; } - @io.grpc.ExperimentalApi("https://github.com/grpc/grpc-java/issues/1901") - @java.lang.Deprecated // Use {@link #getStreamingInputCallMethod()} instead. - public static final io.grpc.MethodDescriptor<io.grpc.testing.compiler.Test.StreamingInputCallRequest, - io.grpc.testing.compiler.Test.StreamingInputCallResponse> METHOD_STREAMING_INPUT_CALL = getStreamingInputCallMethodHelper(); private static volatile io.grpc.MethodDescriptor<io.grpc.testing.compiler.Test.StreamingInputCallRequest, io.grpc.testing.compiler.Test.StreamingInputCallResponse> getStreamingInputCallMethod; - @io.grpc.ExperimentalApi("https://github.com/grpc/grpc-java/issues/1901") public static io.grpc.MethodDescriptor<io.grpc.testing.compiler.Test.StreamingInputCallRequest, io.grpc.testing.compiler.Test.StreamingInputCallResponse> getStreamingInputCallMethod() { - return getStreamingInputCallMethodHelper(); - } - - private static io.grpc.MethodDescriptor<io.grpc.testing.compiler.Test.StreamingInputCallRequest, - io.grpc.testing.compiler.Test.StreamingInputCallResponse> getStreamingInputCallMethodHelper() { io.grpc.MethodDescriptor<io.grpc.testing.compiler.Test.StreamingInputCallRequest, io.grpc.testing.compiler.Test.StreamingInputCallResponse> getStreamingInputCallMethod; if ((getStreamingInputCallMethod = TestServiceGrpc.getStreamingInputCallMethod) == null) { synchronized (TestServiceGrpc.class) { @@ -138,22 +107,12 @@ public final class TestServiceGrpc { } return getStreamingInputCallMethod; } - @io.grpc.ExperimentalApi("https://github.com/grpc/grpc-java/issues/1901") - @java.lang.Deprecated // Use {@link #getFullBidiCallMethod()} instead. - public static final io.grpc.MethodDescriptor<io.grpc.testing.compiler.Test.StreamingOutputCallRequest, - io.grpc.testing.compiler.Test.StreamingOutputCallResponse> METHOD_FULL_BIDI_CALL = getFullBidiCallMethodHelper(); private static volatile io.grpc.MethodDescriptor<io.grpc.testing.compiler.Test.StreamingOutputCallRequest, io.grpc.testing.compiler.Test.StreamingOutputCallResponse> getFullBidiCallMethod; - @io.grpc.ExperimentalApi("https://github.com/grpc/grpc-java/issues/1901") public static io.grpc.MethodDescriptor<io.grpc.testing.compiler.Test.StreamingOutputCallRequest, io.grpc.testing.compiler.Test.StreamingOutputCallResponse> getFullBidiCallMethod() { - return getFullBidiCallMethodHelper(); - } - - private static io.grpc.MethodDescriptor<io.grpc.testing.compiler.Test.StreamingOutputCallRequest, - io.grpc.testing.compiler.Test.StreamingOutputCallResponse> getFullBidiCallMethodHelper() { io.grpc.MethodDescriptor<io.grpc.testing.compiler.Test.StreamingOutputCallRequest, io.grpc.testing.compiler.Test.StreamingOutputCallResponse> getFullBidiCallMethod; if ((getFullBidiCallMethod = TestServiceGrpc.getFullBidiCallMethod) == null) { synchronized (TestServiceGrpc.class) { @@ -174,22 +133,12 @@ public final class TestServiceGrpc { } return getFullBidiCallMethod; } - @io.grpc.ExperimentalApi("https://github.com/grpc/grpc-java/issues/1901") - @java.lang.Deprecated // Use {@link #getHalfBidiCallMethod()} instead. - public static final io.grpc.MethodDescriptor<io.grpc.testing.compiler.Test.StreamingOutputCallRequest, - io.grpc.testing.compiler.Test.StreamingOutputCallResponse> METHOD_HALF_BIDI_CALL = getHalfBidiCallMethodHelper(); private static volatile io.grpc.MethodDescriptor<io.grpc.testing.compiler.Test.StreamingOutputCallRequest, io.grpc.testing.compiler.Test.StreamingOutputCallResponse> getHalfBidiCallMethod; - @io.grpc.ExperimentalApi("https://github.com/grpc/grpc-java/issues/1901") public static io.grpc.MethodDescriptor<io.grpc.testing.compiler.Test.StreamingOutputCallRequest, io.grpc.testing.compiler.Test.StreamingOutputCallResponse> getHalfBidiCallMethod() { - return getHalfBidiCallMethodHelper(); - } - - private static io.grpc.MethodDescriptor<io.grpc.testing.compiler.Test.StreamingOutputCallRequest, - io.grpc.testing.compiler.Test.StreamingOutputCallResponse> getHalfBidiCallMethodHelper() { io.grpc.MethodDescriptor<io.grpc.testing.compiler.Test.StreamingOutputCallRequest, io.grpc.testing.compiler.Test.StreamingOutputCallResponse> getHalfBidiCallMethod; if ((getHalfBidiCallMethod = TestServiceGrpc.getHalfBidiCallMethod) == null) { synchronized (TestServiceGrpc.class) { @@ -210,22 +159,12 @@ public final class TestServiceGrpc { } return getHalfBidiCallMethod; } - @io.grpc.ExperimentalApi("https://github.com/grpc/grpc-java/issues/1901") - @java.lang.Deprecated // Use {@link #getImportMethod()} instead. - public static final io.grpc.MethodDescriptor<io.grpc.testing.compiler.Test.StreamingInputCallRequest, - io.grpc.testing.compiler.Test.StreamingInputCallResponse> METHOD_IMPORT = getImportMethodHelper(); private static volatile io.grpc.MethodDescriptor<io.grpc.testing.compiler.Test.StreamingInputCallRequest, io.grpc.testing.compiler.Test.StreamingInputCallResponse> getImportMethod; - @io.grpc.ExperimentalApi("https://github.com/grpc/grpc-java/issues/1901") public static io.grpc.MethodDescriptor<io.grpc.testing.compiler.Test.StreamingInputCallRequest, io.grpc.testing.compiler.Test.StreamingInputCallResponse> getImportMethod() { - return getImportMethodHelper(); - } - - private static io.grpc.MethodDescriptor<io.grpc.testing.compiler.Test.StreamingInputCallRequest, - io.grpc.testing.compiler.Test.StreamingInputCallResponse> getImportMethodHelper() { io.grpc.MethodDescriptor<io.grpc.testing.compiler.Test.StreamingInputCallRequest, io.grpc.testing.compiler.Test.StreamingInputCallResponse> getImportMethod; if ((getImportMethod = TestServiceGrpc.getImportMethod) == null) { synchronized (TestServiceGrpc.class) { @@ -285,7 +224,7 @@ public final class TestServiceGrpc { */ public void unaryCall(io.grpc.testing.compiler.Test.SimpleRequest request, io.grpc.stub.StreamObserver<io.grpc.testing.compiler.Test.SimpleResponse> responseObserver) { - asyncUnimplementedUnaryCall(getUnaryCallMethodHelper(), responseObserver); + asyncUnimplementedUnaryCall(getUnaryCallMethod(), responseObserver); } /** @@ -296,7 +235,7 @@ public final class TestServiceGrpc { */ public void streamingOutputCall(io.grpc.testing.compiler.Test.StreamingOutputCallRequest request, io.grpc.stub.StreamObserver<io.grpc.testing.compiler.Test.StreamingOutputCallResponse> responseObserver) { - asyncUnimplementedUnaryCall(getStreamingOutputCallMethodHelper(), responseObserver); + asyncUnimplementedUnaryCall(getStreamingOutputCallMethod(), responseObserver); } /** @@ -307,7 +246,7 @@ public final class TestServiceGrpc { */ public io.grpc.stub.StreamObserver<io.grpc.testing.compiler.Test.StreamingInputCallRequest> streamingInputCall( io.grpc.stub.StreamObserver<io.grpc.testing.compiler.Test.StreamingInputCallResponse> responseObserver) { - return asyncUnimplementedStreamingCall(getStreamingInputCallMethodHelper(), responseObserver); + return asyncUnimplementedStreamingCall(getStreamingInputCallMethod(), responseObserver); } /** @@ -319,7 +258,7 @@ public final class TestServiceGrpc { */ public io.grpc.stub.StreamObserver<io.grpc.testing.compiler.Test.StreamingOutputCallRequest> fullBidiCall( io.grpc.stub.StreamObserver<io.grpc.testing.compiler.Test.StreamingOutputCallResponse> responseObserver) { - return asyncUnimplementedStreamingCall(getFullBidiCallMethodHelper(), responseObserver); + return asyncUnimplementedStreamingCall(getFullBidiCallMethod(), responseObserver); } /** @@ -332,7 +271,7 @@ public final class TestServiceGrpc { */ public io.grpc.stub.StreamObserver<io.grpc.testing.compiler.Test.StreamingOutputCallRequest> halfBidiCall( io.grpc.stub.StreamObserver<io.grpc.testing.compiler.Test.StreamingOutputCallResponse> responseObserver) { - return asyncUnimplementedStreamingCall(getHalfBidiCallMethodHelper(), responseObserver); + return asyncUnimplementedStreamingCall(getHalfBidiCallMethod(), responseObserver); } /** @@ -343,48 +282,48 @@ public final class TestServiceGrpc { */ public io.grpc.stub.StreamObserver<io.grpc.testing.compiler.Test.StreamingInputCallRequest> import_( io.grpc.stub.StreamObserver<io.grpc.testing.compiler.Test.StreamingInputCallResponse> responseObserver) { - return asyncUnimplementedStreamingCall(getImportMethodHelper(), responseObserver); + return asyncUnimplementedStreamingCall(getImportMethod(), responseObserver); } @java.lang.Override public final io.grpc.ServerServiceDefinition bindService() { return io.grpc.ServerServiceDefinition.builder(getServiceDescriptor()) .addMethod( - getUnaryCallMethodHelper(), + getUnaryCallMethod(), asyncUnaryCall( new MethodHandlers< io.grpc.testing.compiler.Test.SimpleRequest, io.grpc.testing.compiler.Test.SimpleResponse>( this, METHODID_UNARY_CALL))) .addMethod( - getStreamingOutputCallMethodHelper(), + getStreamingOutputCallMethod(), asyncServerStreamingCall( new MethodHandlers< io.grpc.testing.compiler.Test.StreamingOutputCallRequest, io.grpc.testing.compiler.Test.StreamingOutputCallResponse>( this, METHODID_STREAMING_OUTPUT_CALL))) .addMethod( - getStreamingInputCallMethodHelper(), + getStreamingInputCallMethod(), asyncClientStreamingCall( new MethodHandlers< io.grpc.testing.compiler.Test.StreamingInputCallRequest, io.grpc.testing.compiler.Test.StreamingInputCallResponse>( this, METHODID_STREAMING_INPUT_CALL))) .addMethod( - getFullBidiCallMethodHelper(), + getFullBidiCallMethod(), asyncBidiStreamingCall( new MethodHandlers< io.grpc.testing.compiler.Test.StreamingOutputCallRequest, io.grpc.testing.compiler.Test.StreamingOutputCallResponse>( this, METHODID_FULL_BIDI_CALL))) .addMethod( - getHalfBidiCallMethodHelper(), + getHalfBidiCallMethod(), asyncBidiStreamingCall( new MethodHandlers< io.grpc.testing.compiler.Test.StreamingOutputCallRequest, io.grpc.testing.compiler.Test.StreamingOutputCallResponse>( this, METHODID_HALF_BIDI_CALL))) .addMethod( - getImportMethodHelper(), + getImportMethod(), asyncBidiStreamingCall( new MethodHandlers< io.grpc.testing.compiler.Test.StreamingInputCallRequest, @@ -424,7 +363,7 @@ public final class TestServiceGrpc { public void unaryCall(io.grpc.testing.compiler.Test.SimpleRequest request, io.grpc.stub.StreamObserver<io.grpc.testing.compiler.Test.SimpleResponse> responseObserver) { asyncUnaryCall( - getChannel().newCall(getUnaryCallMethodHelper(), getCallOptions()), request, responseObserver); + getChannel().newCall(getUnaryCallMethod(), getCallOptions()), request, responseObserver); } /** @@ -436,7 +375,7 @@ public final class TestServiceGrpc { public void streamingOutputCall(io.grpc.testing.compiler.Test.StreamingOutputCallRequest request, io.grpc.stub.StreamObserver<io.grpc.testing.compiler.Test.StreamingOutputCallResponse> responseObserver) { asyncServerStreamingCall( - getChannel().newCall(getStreamingOutputCallMethodHelper(), getCallOptions()), request, responseObserver); + getChannel().newCall(getStreamingOutputCallMethod(), getCallOptions()), request, responseObserver); } /** @@ -448,7 +387,7 @@ public final class TestServiceGrpc { public io.grpc.stub.StreamObserver<io.grpc.testing.compiler.Test.StreamingInputCallRequest> streamingInputCall( io.grpc.stub.StreamObserver<io.grpc.testing.compiler.Test.StreamingInputCallResponse> responseObserver) { return asyncClientStreamingCall( - getChannel().newCall(getStreamingInputCallMethodHelper(), getCallOptions()), responseObserver); + getChannel().newCall(getStreamingInputCallMethod(), getCallOptions()), responseObserver); } /** @@ -461,7 +400,7 @@ public final class TestServiceGrpc { public io.grpc.stub.StreamObserver<io.grpc.testing.compiler.Test.StreamingOutputCallRequest> fullBidiCall( io.grpc.stub.StreamObserver<io.grpc.testing.compiler.Test.StreamingOutputCallResponse> responseObserver) { return asyncBidiStreamingCall( - getChannel().newCall(getFullBidiCallMethodHelper(), getCallOptions()), responseObserver); + getChannel().newCall(getFullBidiCallMethod(), getCallOptions()), responseObserver); } /** @@ -475,7 +414,7 @@ public final class TestServiceGrpc { public io.grpc.stub.StreamObserver<io.grpc.testing.compiler.Test.StreamingOutputCallRequest> halfBidiCall( io.grpc.stub.StreamObserver<io.grpc.testing.compiler.Test.StreamingOutputCallResponse> responseObserver) { return asyncBidiStreamingCall( - getChannel().newCall(getHalfBidiCallMethodHelper(), getCallOptions()), responseObserver); + getChannel().newCall(getHalfBidiCallMethod(), getCallOptions()), responseObserver); } /** @@ -487,7 +426,7 @@ public final class TestServiceGrpc { public io.grpc.stub.StreamObserver<io.grpc.testing.compiler.Test.StreamingInputCallRequest> import_( io.grpc.stub.StreamObserver<io.grpc.testing.compiler.Test.StreamingInputCallResponse> responseObserver) { return asyncBidiStreamingCall( - getChannel().newCall(getImportMethodHelper(), getCallOptions()), responseObserver); + getChannel().newCall(getImportMethod(), getCallOptions()), responseObserver); } } @@ -520,7 +459,7 @@ public final class TestServiceGrpc { */ public io.grpc.testing.compiler.Test.SimpleResponse unaryCall(io.grpc.testing.compiler.Test.SimpleRequest request) { return blockingUnaryCall( - getChannel(), getUnaryCallMethodHelper(), getCallOptions(), request); + getChannel(), getUnaryCallMethod(), getCallOptions(), request); } /** @@ -532,7 +471,7 @@ public final class TestServiceGrpc { public java.util.Iterator<io.grpc.testing.compiler.Test.StreamingOutputCallResponse> streamingOutputCall( io.grpc.testing.compiler.Test.StreamingOutputCallRequest request) { return blockingServerStreamingCall( - getChannel(), getStreamingOutputCallMethodHelper(), getCallOptions(), request); + getChannel(), getStreamingOutputCallMethod(), getCallOptions(), request); } } @@ -566,7 +505,7 @@ public final class TestServiceGrpc { public com.google.common.util.concurrent.ListenableFuture<io.grpc.testing.compiler.Test.SimpleResponse> unaryCall( io.grpc.testing.compiler.Test.SimpleRequest request) { return futureUnaryCall( - getChannel().newCall(getUnaryCallMethodHelper(), getCallOptions()), request); + getChannel().newCall(getUnaryCallMethod(), getCallOptions()), request); } } @@ -639,12 +578,12 @@ public final class TestServiceGrpc { result = serviceDescriptor; if (result == null) { serviceDescriptor = result = io.grpc.ServiceDescriptor.newBuilder(SERVICE_NAME) - .addMethod(getUnaryCallMethodHelper()) - .addMethod(getStreamingOutputCallMethodHelper()) - .addMethod(getStreamingInputCallMethodHelper()) - .addMethod(getFullBidiCallMethodHelper()) - .addMethod(getHalfBidiCallMethodHelper()) - .addMethod(getImportMethodHelper()) + .addMethod(getUnaryCallMethod()) + .addMethod(getStreamingOutputCallMethod()) + .addMethod(getStreamingInputCallMethod()) + .addMethod(getFullBidiCallMethod()) + .addMethod(getHalfBidiCallMethod()) + .addMethod(getImportMethod()) .build(); } } diff --git a/compiler/src/testNano/golden/TestService.java.txt b/compiler/src/testNano/golden/TestService.java.txt index 3616ffde0..7809add4c 100644 --- a/compiler/src/testNano/golden/TestService.java.txt +++ b/compiler/src/testNano/golden/TestService.java.txt @@ -34,22 +34,11 @@ public final class TestServiceGrpc { // Static method descriptors that strictly reflect the proto. private static final int ARG_IN_METHOD_UNARY_CALL = 0; private static final int ARG_OUT_METHOD_UNARY_CALL = 1; - @io.grpc.ExperimentalApi("https://github.com/grpc/grpc-java/issues/1901") - @java.lang.Deprecated // Use {@link #getUnaryCallMethod()} instead. - public static final io.grpc.MethodDescriptor<io.grpc.testing.compiler.nano.Test.SimpleRequest, - io.grpc.testing.compiler.nano.Test.SimpleResponse> METHOD_UNARY_CALL = getUnaryCallMethodHelper(); - private static volatile io.grpc.MethodDescriptor<io.grpc.testing.compiler.nano.Test.SimpleRequest, io.grpc.testing.compiler.nano.Test.SimpleResponse> getUnaryCallMethod; - @io.grpc.ExperimentalApi("https://github.com/grpc/grpc-java/issues/1901") public static io.grpc.MethodDescriptor<io.grpc.testing.compiler.nano.Test.SimpleRequest, io.grpc.testing.compiler.nano.Test.SimpleResponse> getUnaryCallMethod() { - return getUnaryCallMethodHelper(); - } - - private static io.grpc.MethodDescriptor<io.grpc.testing.compiler.nano.Test.SimpleRequest, - io.grpc.testing.compiler.nano.Test.SimpleResponse> getUnaryCallMethodHelper() { io.grpc.MethodDescriptor<io.grpc.testing.compiler.nano.Test.SimpleRequest, io.grpc.testing.compiler.nano.Test.SimpleResponse> getUnaryCallMethod; if ((getUnaryCallMethod = TestServiceGrpc.getUnaryCallMethod) == null) { synchronized (TestServiceGrpc.class) { @@ -70,24 +59,14 @@ public final class TestServiceGrpc { } return getUnaryCallMethod; } + private static final int ARG_IN_METHOD_STREAMING_OUTPUT_CALL = 2; private static final int ARG_OUT_METHOD_STREAMING_OUTPUT_CALL = 3; - @io.grpc.ExperimentalApi("https://github.com/grpc/grpc-java/issues/1901") - @java.lang.Deprecated // Use {@link #getStreamingOutputCallMethod()} instead. - public static final io.grpc.MethodDescriptor<io.grpc.testing.compiler.nano.Test.StreamingOutputCallRequest, - io.grpc.testing.compiler.nano.Test.StreamingOutputCallResponse> METHOD_STREAMING_OUTPUT_CALL = getStreamingOutputCallMethodHelper(); - private static volatile io.grpc.MethodDescriptor<io.grpc.testing.compiler.nano.Test.StreamingOutputCallRequest, io.grpc.testing.compiler.nano.Test.StreamingOutputCallResponse> getStreamingOutputCallMethod; - @io.grpc.ExperimentalApi("https://github.com/grpc/grpc-java/issues/1901") public static io.grpc.MethodDescriptor<io.grpc.testing.compiler.nano.Test.StreamingOutputCallRequest, io.grpc.testing.compiler.nano.Test.StreamingOutputCallResponse> getStreamingOutputCallMethod() { - return getStreamingOutputCallMethodHelper(); - } - - private static io.grpc.MethodDescriptor<io.grpc.testing.compiler.nano.Test.StreamingOutputCallRequest, - io.grpc.testing.compiler.nano.Test.StreamingOutputCallResponse> getStreamingOutputCallMethodHelper() { io.grpc.MethodDescriptor<io.grpc.testing.compiler.nano.Test.StreamingOutputCallRequest, io.grpc.testing.compiler.nano.Test.StreamingOutputCallResponse> getStreamingOutputCallMethod; if ((getStreamingOutputCallMethod = TestServiceGrpc.getStreamingOutputCallMethod) == null) { synchronized (TestServiceGrpc.class) { @@ -108,24 +87,14 @@ public final class TestServiceGrpc { } return getStreamingOutputCallMethod; } + private static final int ARG_IN_METHOD_STREAMING_INPUT_CALL = 4; private static final int ARG_OUT_METHOD_STREAMING_INPUT_CALL = 5; - @io.grpc.ExperimentalApi("https://github.com/grpc/grpc-java/issues/1901") - @java.lang.Deprecated // Use {@link #getStreamingInputCallMethod()} instead. - public static final io.grpc.MethodDescriptor<io.grpc.testing.compiler.nano.Test.StreamingInputCallRequest, - io.grpc.testing.compiler.nano.Test.StreamingInputCallResponse> METHOD_STREAMING_INPUT_CALL = getStreamingInputCallMethodHelper(); - private static volatile io.grpc.MethodDescriptor<io.grpc.testing.compiler.nano.Test.StreamingInputCallRequest, io.grpc.testing.compiler.nano.Test.StreamingInputCallResponse> getStreamingInputCallMethod; - @io.grpc.ExperimentalApi("https://github.com/grpc/grpc-java/issues/1901") public static io.grpc.MethodDescriptor<io.grpc.testing.compiler.nano.Test.StreamingInputCallRequest, io.grpc.testing.compiler.nano.Test.StreamingInputCallResponse> getStreamingInputCallMethod() { - return getStreamingInputCallMethodHelper(); - } - - private static io.grpc.MethodDescriptor<io.grpc.testing.compiler.nano.Test.StreamingInputCallRequest, - io.grpc.testing.compiler.nano.Test.StreamingInputCallResponse> getStreamingInputCallMethodHelper() { io.grpc.MethodDescriptor<io.grpc.testing.compiler.nano.Test.StreamingInputCallRequest, io.grpc.testing.compiler.nano.Test.StreamingInputCallResponse> getStreamingInputCallMethod; if ((getStreamingInputCallMethod = TestServiceGrpc.getStreamingInputCallMethod) == null) { synchronized (TestServiceGrpc.class) { @@ -146,24 +115,14 @@ public final class TestServiceGrpc { } return getStreamingInputCallMethod; } + private static final int ARG_IN_METHOD_FULL_BIDI_CALL = 6; private static final int ARG_OUT_METHOD_FULL_BIDI_CALL = 7; - @io.grpc.ExperimentalApi("https://github.com/grpc/grpc-java/issues/1901") - @java.lang.Deprecated // Use {@link #getFullBidiCallMethod()} instead. - public static final io.grpc.MethodDescriptor<io.grpc.testing.compiler.nano.Test.StreamingOutputCallRequest, - io.grpc.testing.compiler.nano.Test.StreamingOutputCallResponse> METHOD_FULL_BIDI_CALL = getFullBidiCallMethodHelper(); - private static volatile io.grpc.MethodDescriptor<io.grpc.testing.compiler.nano.Test.StreamingOutputCallRequest, io.grpc.testing.compiler.nano.Test.StreamingOutputCallResponse> getFullBidiCallMethod; - @io.grpc.ExperimentalApi("https://github.com/grpc/grpc-java/issues/1901") public static io.grpc.MethodDescriptor<io.grpc.testing.compiler.nano.Test.StreamingOutputCallRequest, io.grpc.testing.compiler.nano.Test.StreamingOutputCallResponse> getFullBidiCallMethod() { - return getFullBidiCallMethodHelper(); - } - - private static io.grpc.MethodDescriptor<io.grpc.testing.compiler.nano.Test.StreamingOutputCallRequest, - io.grpc.testing.compiler.nano.Test.StreamingOutputCallResponse> getFullBidiCallMethodHelper() { io.grpc.MethodDescriptor<io.grpc.testing.compiler.nano.Test.StreamingOutputCallRequest, io.grpc.testing.compiler.nano.Test.StreamingOutputCallResponse> getFullBidiCallMethod; if ((getFullBidiCallMethod = TestServiceGrpc.getFullBidiCallMethod) == null) { synchronized (TestServiceGrpc.class) { @@ -184,24 +143,14 @@ public final class TestServiceGrpc { } return getFullBidiCallMethod; } + private static final int ARG_IN_METHOD_HALF_BIDI_CALL = 8; private static final int ARG_OUT_METHOD_HALF_BIDI_CALL = 9; - @io.grpc.ExperimentalApi("https://github.com/grpc/grpc-java/issues/1901") - @java.lang.Deprecated // Use {@link #getHalfBidiCallMethod()} instead. - public static final io.grpc.MethodDescriptor<io.grpc.testing.compiler.nano.Test.StreamingOutputCallRequest, - io.grpc.testing.compiler.nano.Test.StreamingOutputCallResponse> METHOD_HALF_BIDI_CALL = getHalfBidiCallMethodHelper(); - private static volatile io.grpc.MethodDescriptor<io.grpc.testing.compiler.nano.Test.StreamingOutputCallRequest, io.grpc.testing.compiler.nano.Test.StreamingOutputCallResponse> getHalfBidiCallMethod; - @io.grpc.ExperimentalApi("https://github.com/grpc/grpc-java/issues/1901") public static io.grpc.MethodDescriptor<io.grpc.testing.compiler.nano.Test.StreamingOutputCallRequest, io.grpc.testing.compiler.nano.Test.StreamingOutputCallResponse> getHalfBidiCallMethod() { - return getHalfBidiCallMethodHelper(); - } - - private static io.grpc.MethodDescriptor<io.grpc.testing.compiler.nano.Test.StreamingOutputCallRequest, - io.grpc.testing.compiler.nano.Test.StreamingOutputCallResponse> getHalfBidiCallMethodHelper() { io.grpc.MethodDescriptor<io.grpc.testing.compiler.nano.Test.StreamingOutputCallRequest, io.grpc.testing.compiler.nano.Test.StreamingOutputCallResponse> getHalfBidiCallMethod; if ((getHalfBidiCallMethod = TestServiceGrpc.getHalfBidiCallMethod) == null) { synchronized (TestServiceGrpc.class) { @@ -222,24 +171,14 @@ public final class TestServiceGrpc { } return getHalfBidiCallMethod; } + private static final int ARG_IN_METHOD_IMPORT = 10; private static final int ARG_OUT_METHOD_IMPORT = 11; - @io.grpc.ExperimentalApi("https://github.com/grpc/grpc-java/issues/1901") - @java.lang.Deprecated // Use {@link #getImportMethod()} instead. - public static final io.grpc.MethodDescriptor<io.grpc.testing.compiler.nano.Test.StreamingInputCallRequest, - io.grpc.testing.compiler.nano.Test.StreamingInputCallResponse> METHOD_IMPORT = getImportMethodHelper(); - private static volatile io.grpc.MethodDescriptor<io.grpc.testing.compiler.nano.Test.StreamingInputCallRequest, io.grpc.testing.compiler.nano.Test.StreamingInputCallResponse> getImportMethod; - @io.grpc.ExperimentalApi("https://github.com/grpc/grpc-java/issues/1901") public static io.grpc.MethodDescriptor<io.grpc.testing.compiler.nano.Test.StreamingInputCallRequest, io.grpc.testing.compiler.nano.Test.StreamingInputCallResponse> getImportMethod() { - return getImportMethodHelper(); - } - - private static io.grpc.MethodDescriptor<io.grpc.testing.compiler.nano.Test.StreamingInputCallRequest, - io.grpc.testing.compiler.nano.Test.StreamingInputCallResponse> getImportMethodHelper() { io.grpc.MethodDescriptor<io.grpc.testing.compiler.nano.Test.StreamingInputCallRequest, io.grpc.testing.compiler.nano.Test.StreamingInputCallResponse> getImportMethod; if ((getImportMethod = TestServiceGrpc.getImportMethod) == null) { synchronized (TestServiceGrpc.class) { @@ -356,7 +295,7 @@ public final class TestServiceGrpc { */ public void unaryCall(io.grpc.testing.compiler.nano.Test.SimpleRequest request, io.grpc.stub.StreamObserver<io.grpc.testing.compiler.nano.Test.SimpleResponse> responseObserver) { - asyncUnimplementedUnaryCall(getUnaryCallMethodHelper(), responseObserver); + asyncUnimplementedUnaryCall(getUnaryCallMethod(), responseObserver); } /** @@ -367,7 +306,7 @@ public final class TestServiceGrpc { */ public void streamingOutputCall(io.grpc.testing.compiler.nano.Test.StreamingOutputCallRequest request, io.grpc.stub.StreamObserver<io.grpc.testing.compiler.nano.Test.StreamingOutputCallResponse> responseObserver) { - asyncUnimplementedUnaryCall(getStreamingOutputCallMethodHelper(), responseObserver); + asyncUnimplementedUnaryCall(getStreamingOutputCallMethod(), responseObserver); } /** @@ -378,7 +317,7 @@ public final class TestServiceGrpc { */ public io.grpc.stub.StreamObserver<io.grpc.testing.compiler.nano.Test.StreamingInputCallRequest> streamingInputCall( io.grpc.stub.StreamObserver<io.grpc.testing.compiler.nano.Test.StreamingInputCallResponse> responseObserver) { - return asyncUnimplementedStreamingCall(getStreamingInputCallMethodHelper(), responseObserver); + return asyncUnimplementedStreamingCall(getStreamingInputCallMethod(), responseObserver); } /** @@ -390,7 +329,7 @@ public final class TestServiceGrpc { */ public io.grpc.stub.StreamObserver<io.grpc.testing.compiler.nano.Test.StreamingOutputCallRequest> fullBidiCall( io.grpc.stub.StreamObserver<io.grpc.testing.compiler.nano.Test.StreamingOutputCallResponse> responseObserver) { - return asyncUnimplementedStreamingCall(getFullBidiCallMethodHelper(), responseObserver); + return asyncUnimplementedStreamingCall(getFullBidiCallMethod(), responseObserver); } /** @@ -403,7 +342,7 @@ public final class TestServiceGrpc { */ public io.grpc.stub.StreamObserver<io.grpc.testing.compiler.nano.Test.StreamingOutputCallRequest> halfBidiCall( io.grpc.stub.StreamObserver<io.grpc.testing.compiler.nano.Test.StreamingOutputCallResponse> responseObserver) { - return asyncUnimplementedStreamingCall(getHalfBidiCallMethodHelper(), responseObserver); + return asyncUnimplementedStreamingCall(getHalfBidiCallMethod(), responseObserver); } /** @@ -414,48 +353,48 @@ public final class TestServiceGrpc { */ public io.grpc.stub.StreamObserver<io.grpc.testing.compiler.nano.Test.StreamingInputCallRequest> import_( io.grpc.stub.StreamObserver<io.grpc.testing.compiler.nano.Test.StreamingInputCallResponse> responseObserver) { - return asyncUnimplementedStreamingCall(getImportMethodHelper(), responseObserver); + return asyncUnimplementedStreamingCall(getImportMethod(), responseObserver); } @java.lang.Override public final io.grpc.ServerServiceDefinition bindService() { return io.grpc.ServerServiceDefinition.builder(getServiceDescriptor()) .addMethod( - getUnaryCallMethodHelper(), + getUnaryCallMethod(), asyncUnaryCall( new MethodHandlers< io.grpc.testing.compiler.nano.Test.SimpleRequest, io.grpc.testing.compiler.nano.Test.SimpleResponse>( this, METHODID_UNARY_CALL))) .addMethod( - getStreamingOutputCallMethodHelper(), + getStreamingOutputCallMethod(), asyncServerStreamingCall( new MethodHandlers< io.grpc.testing.compiler.nano.Test.StreamingOutputCallRequest, io.grpc.testing.compiler.nano.Test.StreamingOutputCallResponse>( this, METHODID_STREAMING_OUTPUT_CALL))) .addMethod( - getStreamingInputCallMethodHelper(), + getStreamingInputCallMethod(), asyncClientStreamingCall( new MethodHandlers< io.grpc.testing.compiler.nano.Test.StreamingInputCallRequest, io.grpc.testing.compiler.nano.Test.StreamingInputCallResponse>( this, METHODID_STREAMING_INPUT_CALL))) .addMethod( - getFullBidiCallMethodHelper(), + getFullBidiCallMethod(), asyncBidiStreamingCall( new MethodHandlers< io.grpc.testing.compiler.nano.Test.StreamingOutputCallRequest, io.grpc.testing.compiler.nano.Test.StreamingOutputCallResponse>( this, METHODID_FULL_BIDI_CALL))) .addMethod( - getHalfBidiCallMethodHelper(), + getHalfBidiCallMethod(), asyncBidiStreamingCall( new MethodHandlers< io.grpc.testing.compiler.nano.Test.StreamingOutputCallRequest, io.grpc.testing.compiler.nano.Test.StreamingOutputCallResponse>( this, METHODID_HALF_BIDI_CALL))) .addMethod( - getImportMethodHelper(), + getImportMethod(), asyncBidiStreamingCall( new MethodHandlers< io.grpc.testing.compiler.nano.Test.StreamingInputCallRequest, @@ -495,7 +434,7 @@ public final class TestServiceGrpc { public void unaryCall(io.grpc.testing.compiler.nano.Test.SimpleRequest request, io.grpc.stub.StreamObserver<io.grpc.testing.compiler.nano.Test.SimpleResponse> responseObserver) { asyncUnaryCall( - getChannel().newCall(getUnaryCallMethodHelper(), getCallOptions()), request, responseObserver); + getChannel().newCall(getUnaryCallMethod(), getCallOptions()), request, responseObserver); } /** @@ -507,7 +446,7 @@ public final class TestServiceGrpc { public void streamingOutputCall(io.grpc.testing.compiler.nano.Test.StreamingOutputCallRequest request, io.grpc.stub.StreamObserver<io.grpc.testing.compiler.nano.Test.StreamingOutputCallResponse> responseObserver) { asyncServerStreamingCall( - getChannel().newCall(getStreamingOutputCallMethodHelper(), getCallOptions()), request, responseObserver); + getChannel().newCall(getStreamingOutputCallMethod(), getCallOptions()), request, responseObserver); } /** @@ -519,7 +458,7 @@ public final class TestServiceGrpc { public io.grpc.stub.StreamObserver<io.grpc.testing.compiler.nano.Test.StreamingInputCallRequest> streamingInputCall( io.grpc.stub.StreamObserver<io.grpc.testing.compiler.nano.Test.StreamingInputCallResponse> responseObserver) { return asyncClientStreamingCall( - getChannel().newCall(getStreamingInputCallMethodHelper(), getCallOptions()), responseObserver); + getChannel().newCall(getStreamingInputCallMethod(), getCallOptions()), responseObserver); } /** @@ -532,7 +471,7 @@ public final class TestServiceGrpc { public io.grpc.stub.StreamObserver<io.grpc.testing.compiler.nano.Test.StreamingOutputCallRequest> fullBidiCall( io.grpc.stub.StreamObserver<io.grpc.testing.compiler.nano.Test.StreamingOutputCallResponse> responseObserver) { return asyncBidiStreamingCall( - getChannel().newCall(getFullBidiCallMethodHelper(), getCallOptions()), responseObserver); + getChannel().newCall(getFullBidiCallMethod(), getCallOptions()), responseObserver); } /** @@ -546,7 +485,7 @@ public final class TestServiceGrpc { public io.grpc.stub.StreamObserver<io.grpc.testing.compiler.nano.Test.StreamingOutputCallRequest> halfBidiCall( io.grpc.stub.StreamObserver<io.grpc.testing.compiler.nano.Test.StreamingOutputCallResponse> responseObserver) { return asyncBidiStreamingCall( - getChannel().newCall(getHalfBidiCallMethodHelper(), getCallOptions()), responseObserver); + getChannel().newCall(getHalfBidiCallMethod(), getCallOptions()), responseObserver); } /** @@ -558,7 +497,7 @@ public final class TestServiceGrpc { public io.grpc.stub.StreamObserver<io.grpc.testing.compiler.nano.Test.StreamingInputCallRequest> import_( io.grpc.stub.StreamObserver<io.grpc.testing.compiler.nano.Test.StreamingInputCallResponse> responseObserver) { return asyncBidiStreamingCall( - getChannel().newCall(getImportMethodHelper(), getCallOptions()), responseObserver); + getChannel().newCall(getImportMethod(), getCallOptions()), responseObserver); } } @@ -591,7 +530,7 @@ public final class TestServiceGrpc { */ public io.grpc.testing.compiler.nano.Test.SimpleResponse unaryCall(io.grpc.testing.compiler.nano.Test.SimpleRequest request) { return blockingUnaryCall( - getChannel(), getUnaryCallMethodHelper(), getCallOptions(), request); + getChannel(), getUnaryCallMethod(), getCallOptions(), request); } /** @@ -603,7 +542,7 @@ public final class TestServiceGrpc { public java.util.Iterator<io.grpc.testing.compiler.nano.Test.StreamingOutputCallResponse> streamingOutputCall( io.grpc.testing.compiler.nano.Test.StreamingOutputCallRequest request) { return blockingServerStreamingCall( - getChannel(), getStreamingOutputCallMethodHelper(), getCallOptions(), request); + getChannel(), getStreamingOutputCallMethod(), getCallOptions(), request); } } @@ -637,7 +576,7 @@ public final class TestServiceGrpc { public com.google.common.util.concurrent.ListenableFuture<io.grpc.testing.compiler.nano.Test.SimpleResponse> unaryCall( io.grpc.testing.compiler.nano.Test.SimpleRequest request) { return futureUnaryCall( - getChannel().newCall(getUnaryCallMethodHelper(), getCallOptions()), request); + getChannel().newCall(getUnaryCallMethod(), getCallOptions()), request); } } @@ -710,12 +649,12 @@ public final class TestServiceGrpc { result = serviceDescriptor; if (result == null) { serviceDescriptor = result = io.grpc.ServiceDescriptor.newBuilder(SERVICE_NAME) - .addMethod(getUnaryCallMethodHelper()) - .addMethod(getStreamingOutputCallMethodHelper()) - .addMethod(getStreamingInputCallMethodHelper()) - .addMethod(getFullBidiCallMethodHelper()) - .addMethod(getHalfBidiCallMethodHelper()) - .addMethod(getImportMethodHelper()) + .addMethod(getUnaryCallMethod()) + .addMethod(getStreamingOutputCallMethod()) + .addMethod(getStreamingInputCallMethod()) + .addMethod(getFullBidiCallMethod()) + .addMethod(getHalfBidiCallMethod()) + .addMethod(getImportMethod()) .build(); } } |