You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@beam.apache.org by GitBox <gi...@apache.org> on 2022/03/26 00:55:51 UTC

[GitHub] [beam] tvalentyn commented on a change in pull request #13986: [BEAM-11779] Use shuffle_mode=appliance when using dataflow runner v2

tvalentyn commented on a change in pull request #13986:
URL: https://github.com/apache/beam/pull/13986#discussion_r835687617



##########
File path: .test-infra/jenkins/job_LoadTests_ParDo_Python.groovy
##########
@@ -131,7 +131,8 @@ def addStreamingOptions(test) {
     // Use the new Dataflow runner, which offers improved efficiency of Dataflow jobs.
     // See https://cloud.google.com/dataflow/docs/guides/deploying-a-pipeline#dataflow-runner-v2
     // for more details.
-    experiments: 'use_runner_v2',
+    // TODO(BEAM-11779) remove shuffle_mode=appliance with runner v2 once issue is resolved.
+    experiments: 'use_runner_v2, shuffle_mode=appliance',

Review comment:
       this has no effect as it should be a separate experiment. should this change be reverted now in it's entirety? 




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@beam.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org