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

 ##########
 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'm not sure we should add these here because that's what we used to do and 
it was inconsistent and hard to maintain. What we want to do, is to not discard 
those options but still warn about them not being parsed in the Python SDK. 
This will allow us to still use them in the Runner code.

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

Reply via email to