tvalentyn commented on a change in pull request #16372:
URL: https://github.com/apache/beam/pull/16372#discussion_r776441707
##########
File path: sdks/python/apache_beam/options/pipeline_options_validator.py
##########
@@ -365,3 +368,16 @@ def validate_environment_options(self, view):
self._validate_error(
self.ERR_INVALID_ENVIRONMENT, 'environment_config', 'LOOPBACK'))
return errors
+
+ def validate_repeatable_argument_passed_as_list(self, view, arg_name):
+ """Validates that repeatable PipelineOptions like dataflow_service_options
Review comment:
nit: comment wording. Suggestion:
```
Validates that repeatable PipelineOptions like dataflow_service_options
or experiments are specified as a list when set programmatically. This
way, users do not inadvertently specify them as a string, which triggers
incorrect implicit string -> list conversion.
```
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
