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/11 16:58:17 UTC

[GitHub] [beam] mxm edited a comment on pull request #12385: [BEAM-10527] Migrate Flink and Spark tests to pytest.

mxm edited a comment on pull request #12385:
URL: https://github.com/apache/beam/pull/12385#issuecomment-672090815


   > > 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?
   
   I think it makes sense. Especially for error reporting by having dedicated argument parsers for all environments parameters.


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