ananvay commented on a change in pull request #11466:
URL: https://github.com/apache/beam/pull/11466#discussion_r411645755
##########
File path: sdks/python/apache_beam/runners/dataflow/dataflow_runner.py
##########
@@ -1182,6 +1188,11 @@ def run_Read(self, transform_node, options):
raise ValueError(
'BigQuery source is not currently available for use '
'in streaming pipelines.')
+ debug_options = options.view_as(DebugOptions)
+ use_fn_api = (debug_options.experiments and
+ 'beam_fn_api' in debug_options.experiments)
+ if use_fn_api:
Review comment:
In general, yes, but the only support way of batch fnapi is with UW --
they shouldn't be trying fnapi batch jobs without it. That said, perhaps it
makes sense to check both separately (just to ensure that the --use_runner_v2
also independently triggers this check, just in case someone in the future
decides to accidentally modify something)?
----------------------------------------------------------------
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]
