ibzib commented on a change in pull request #11052: [BEAM-9446] Add missing
parallelism and execution mode args.
URL: https://github.com/apache/beam/pull/11052#discussion_r392331379
##########
File path: sdks/python/apache_beam/options/pipeline_options.py
##########
@@ -1075,6 +1075,22 @@ def _add_argparse_args(cls, parser):
' directly, rather than starting up a job server.'
' Only applies when flink_master is set to a'
' cluster address. Requires Python 3.6+.')
+ parser.add_argument(
+ '--parallelism',
+ default=-1,
+ type=int,
+ help='The degree of parallelism to be used when distributing '
+ 'operations onto workers. If the parallelism is not set, the '
+ 'configured Flink default is used, or 1 if none can be found.'
+ )
+ parser.add_argument(
+ '--execution_mode_for_batch',
Review comment:
> I think we should continue to discard invalid options when the client is
aware of the full set of options and can perform the validation
Agreed, we would keep options only when we can't be sure (so just the uber
jar job server for now).
----------------------------------------------------------------
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:
[email protected]
With regards,
Apache Git Services