This is an automated email from the ASF dual-hosted git repository.
xinyu 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 16b7b98 [BEAM-12757]: Remove call for Json Dag in Portable
PipelineRunner as it's not supported yet (#15332)
16b7b98 is described below
commit 16b7b98f7d26ac1dfb341ec756bb38c8b783cd3d
Author: Pawas Chhokra <[email protected]>
AuthorDate: Fri Aug 13 15:09:22 2021 -0700
[BEAM-12757]: Remove call for Json Dag in Portable PipelineRunner as it's
not supported yet (#15332)
---
.../src/main/java/org/apache/beam/runners/samza/SamzaRunner.java | 4 ----
1 file changed, 4 deletions(-)
diff --git
a/runners/samza/src/main/java/org/apache/beam/runners/samza/SamzaRunner.java
b/runners/samza/src/main/java/org/apache/beam/runners/samza/SamzaRunner.java
index 910f1aa..669c0c1 100644
--- a/runners/samza/src/main/java/org/apache/beam/runners/samza/SamzaRunner.java
+++ b/runners/samza/src/main/java/org/apache/beam/runners/samza/SamzaRunner.java
@@ -86,13 +86,9 @@ public class SamzaRunner extends
PipelineRunner<SamzaPipelineResult> {
final String dotGraph = PipelineDotRenderer.toDotString(pipeline);
LOG.info("Portable pipeline to run DOT graph:\n{}", dotGraph);
- final String jsonGraph = PipelineJsonRenderer.toJsonString(pipeline);
- LOG.info("Portable pipeline to run JSON graph:\n{}", jsonGraph);
-
final ConfigBuilder configBuilder = new ConfigBuilder(options);
SamzaPortablePipelineTranslator.createConfig(pipeline, configBuilder,
options);
configBuilder.put(BEAM_DOT_GRAPH, dotGraph);
- configBuilder.put(BEAM_JSON_GRAPH, jsonGraph);
final Config config = configBuilder.build();
options.setConfigOverride(config);