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

lcwik pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/beam.git


The following commit(s) were added to refs/heads/master by this push:
     new 151ae67  Made replaceTransforms protected so that DataflowRunner 
subclass can properly setup overrides.
151ae67 is described below

commit 151ae6792a95f8093e1c798660cac67a6423f587
Author: Jeremie Lenfant-Engelmann <[email protected]>
AuthorDate: Thu Dec 21 09:24:36 2017 -0800

    Made replaceTransforms protected so that DataflowRunner subclass can 
properly
    setup overrides.
---
 .../main/java/org/apache/beam/runners/dataflow/DataflowRunner.java    | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git 
a/runners/google-cloud-dataflow-java/src/main/java/org/apache/beam/runners/dataflow/DataflowRunner.java
 
b/runners/google-cloud-dataflow-java/src/main/java/org/apache/beam/runners/dataflow/DataflowRunner.java
index 3684b47..7b9f1c5 100644
--- 
a/runners/google-cloud-dataflow-java/src/main/java/org/apache/beam/runners/dataflow/DataflowRunner.java
+++ 
b/runners/google-cloud-dataflow-java/src/main/java/org/apache/beam/runners/dataflow/DataflowRunner.java
@@ -834,8 +834,10 @@ public class DataflowRunner extends 
PipelineRunner<DataflowPipelineJob> {
         PropertyNames.ENVIRONMENT_VERSION_JOB_TYPE_KEY, jobType);
   }
 
+  // This method is protected to allow a Google internal subclass to properly
+  // setup overrides.
   @VisibleForTesting
-  void replaceTransforms(Pipeline pipeline) {
+  protected void replaceTransforms(Pipeline pipeline) {
     boolean streaming = options.isStreaming() || 
containsUnboundedPCollection(pipeline);
     // Ensure all outputs of all reads are consumed before potentially 
replacing any
     // Read PTransforms

-- 
To stop receiving notification emails like this one, please contact
['"[email protected]" <[email protected]>'].

Reply via email to