subject:"\[GitHub\] \[beam\] ibzib commented on a change in pull request #12576\: \[BEAM\-10671\] Add environment configuration fields as first\-class pipeline options."

[GitHub] [beam] ibzib commented on a change in pull request #12576: [BEAM-10671] Add environment configuration fields as first-class pipeline options.

2020-09-18 Thread GitBox

ibzib commented on a change in pull request #12576: URL: https://github.com/apache/beam/pull/12576#discussion_r491073811 ## File path: sdks/python/apache_beam/options/pipeline_options_validator_test.py ## @@ -613,31 +615,29 @@ def test_environment_options(self): {

[GitHub] [beam] ibzib commented on a change in pull request #12576: [BEAM-10671] Add environment configuration fields as first-class pipeline options.

2020-09-15 Thread GitBox

ibzib commented on a change in pull request #12576: URL: https://github.com/apache/beam/pull/12576#discussion_r488839465 ## File path: sdks/python/apache_beam/options/pipeline_options.py ## @@ -1044,7 +1044,12 @@ def _add_argparse_args(cls, parser): 'form {"os":