Dedupe doc comments, and slot placeholder for Java

Author: ryapric

protobuf-grpc/pb/echo/v1/echo_pb2_grpc.py

@@ -6,7 +6,12 @@
 
 
 class EchoServiceStub(object):
-    """Service that shows a few ways to work with protobufs and gRPC
+    """TODO: once I figure out how Java works lol
+    // You also need to specify any other appropriate static options for other
+    // languages; here's where you specify the Java package name, for example
+    option java_package = "org.opensourcecorp.workshops.protobuf_grpc.echo.v1";
+
+    Service that shows a few ways to work with protobufs and gRPC
     """
 
     def __init__(self, channel):
@@ -23,7 +28,12 @@ def __init__(self, channel):
 
 
 class EchoServiceServicer(object):
-    """Service that shows a few ways to work with protobufs and gRPC
+    """TODO: once I figure out how Java works lol
+    // You also need to specify any other appropriate static options for other
+    // languages; here's where you specify the Java package name, for example
+    option java_package = "org.opensourcecorp.workshops.protobuf_grpc.echo.v1";
+
+    Service that shows a few ways to work with protobufs and gRPC
     """
 
     def Echo(self, request, context):
@@ -48,7 +58,12 @@ def add_EchoServiceServicer_to_server(servicer, server):
 
  # This class is part of an EXPERIMENTAL API.
 class EchoService(object):
-    """Service that shows a few ways to work with protobufs and gRPC
+    """TODO: once I figure out how Java works lol
+    // You also need to specify any other appropriate static options for other
+    // languages; here's where you specify the Java package name, for example
+    option java_package = "org.opensourcecorp.workshops.protobuf_grpc.echo.v1";
+
+    Service that shows a few ways to work with protobufs and gRPC
     """
 
     @staticmethod

protobuf-grpc/pb/employees/v1/employees.pb.go

@@ -6,8 +6,7 @@
 
 
 class EmployeesServiceStub(object):
-    """Service that shows a few ways to work with protobufs and gRPC
-    """
+    """Missing associated documentation comment in .proto file."""
 
     def __init__(self, channel):
         """Constructor.
@@ -28,8 +27,7 @@ def __init__(self, channel):
 
 
 class EmployeesServiceServicer(object):
-    """Service that shows a few ways to work with protobufs and gRPC
-    """
+    """Missing associated documentation comment in .proto file."""
 
     def GetEmployee(self, request, context):
         """Missing associated documentation comment in .proto file."""
@@ -64,8 +62,7 @@ def add_EmployeesServiceServicer_to_server(servicer, server):
 
  # This class is part of an EXPERIMENTAL API.
 class EmployeesService(object):
-    """Service that shows a few ways to work with protobufs and gRPC
-    """
+    """Missing associated documentation comment in .proto file."""
 
     @staticmethod
     def GetEmployee(request,

protobuf-grpc/pb/employees/v1/employees_grpc.pb.go

@@ -8,6 +8,11 @@ package echo.v1;
 // directory structure that your protoc invocation creates!
 option go_package = "github.com/opensourcecorp/workshops/protobuf-grpc/pb/echo/v1";
 
+// TODO: once I figure out how Java works lol
+// // You also need to specify any other appropriate static options for other
+// // languages; here's where you specify the Java package name, for example
+// option java_package = "org.opensourcecorp.workshops.protobuf_grpc.echo.v1";
+
 // Service that shows a few ways to work with protobufs and gRPC
 service EchoService {
   rpc Echo(EchoRequest) returns (EchoResponse);

protobuf-grpc/pb/employees/v1/employees_pb2_grpc.py

@@ -1,21 +1,21 @@
 syntax = "proto3";
 
-// Proto packages should match the directory structure in your tree. The version
-// suffix is considered best-practice
 package employees.v1;
 
-// So the Go output knows its own import path -- this needs to match the output
-// directory structure that your protoc invocation creates!
 option go_package = "github.com/opensourcecorp/workshops/protobuf-grpc/pb/employees/v1";
+// TODO:
+// option java_package = "org.opensourcecorp.workshops.protobuf_grpc.employees.v1";
 
 // Allow for annotations/options to allow for HTTP transcoding of gRPC requests.
-// You can spec & gen these just fine without, but you'll need to use the
-// grpc-gateway plugin to take advantage of these annotations
+// You can spec & generate these protobufs just fine without, but to actually
+// use them, you'll need to use the grpc-gateway plugin to take advantage of
+// these annotations
 import "google/api/annotations.proto";
 
-// Service that shows a few ways to work with protobufs and gRPC
 service EmployeesService {
   rpc GetEmployee(GetEmployeeRequest) returns (GetEmployeeResponse) {
+    // This is where you use those imported annotations above -- this maps GET
+    // requests over HTTP to the path specified, for example
     option (google.api.http) = {
       get: "/employees/v1/get_employee/{short_name}"
     };