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 2020/08/10 21:48:22 UTC

[GitHub] [beam] ibzib commented on pull request #12385: [BEAM-10527] Migrate Flink and Spark tests to pytest.

ibzib commented on pull request #12385:
URL: https://github.com/apache/beam/pull/12385#issuecomment-671606734


   > > I have already spent a long time trying to fix quotes, so I can't help but wondering: why do we need flinkCompatibilityMatrixPROCESS in the first place, when it is not being run anywhere? If it's important, shouldn't we add it to some postcommit?
   > 
   > Another solution I had in mind was reworking the `--environment_config` option. JSON blobs are unwieldy, and overloading the `--environment_config` option is confusing to the user. We could make each field in the PROCESS `--environment_config` blob its own argument, and then reject these arguments when `environment_type != PROCESS`.
   
   @mxm what do you think about https://issues.apache.org/jira/browse/BEAM-10671?


----------------------------------------------------------------
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.

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