Clarify side input name and remove redundant pcollection reference.

Project: http://git-wip-us.apache.org/repos/asf/beam/repo
Commit: http://git-wip-us.apache.org/repos/asf/beam/commit/c04e8abd
Tree: http://git-wip-us.apache.org/repos/asf/beam/tree/c04e8abd
Diff: http://git-wip-us.apache.org/repos/asf/beam/diff/c04e8abd

Branch: refs/heads/master
Commit: c04e8abd41575a4e819c3f18dabff921338f55d8
Parents: a0dcd3f
Author: Robert Bradshaw <rober...@gmail.com>
Authored: Fri Feb 24 16:31:05 2017 -0800
Committer: Robert Bradshaw <rober...@gmail.com>
Committed: Mon Feb 27 17:46:01 2017 -0800

----------------------------------------------------------------------
 sdks/common/runner-api/src/main/proto/beam_runner_api.proto | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/beam/blob/c04e8abd/sdks/common/runner-api/src/main/proto/beam_runner_api.proto
----------------------------------------------------------------------
diff --git a/sdks/common/runner-api/src/main/proto/beam_runner_api.proto 
b/sdks/common/runner-api/src/main/proto/beam_runner_api.proto
index ad6d0cb..258c278 100644
--- a/sdks/common/runner-api/src/main/proto/beam_runner_api.proto
+++ b/sdks/common/runner-api/src/main/proto/beam_runner_api.proto
@@ -576,12 +576,10 @@ message TimestampTransform {
 // A specification for how to "side input" a PCollection.
 message SideInput {
 
-  // (Required) A local name for this side input, as embedded in a serialized 
UDF.
+  // (Required) A local name for this side input, as interpreted by its
+  // parent PTransform (and/or its PTransform's UDFs).
   string name = 1;
 
-  // (Required) The pipeline-scoped unique id of the PCollection to be side 
input.
-  string pcollection_id = 2;
-
   // (Required) URN of the access pattern required by the `view_fn` to present
   // the desired SDK-specific interface to a UDF.
   //

Reply via email to