satybald commented on a change in pull request #15536:
URL: https://github.com/apache/beam/pull/15536#discussion_r713428062
##########
File path: sdks/python/apache_beam/io/gcp/bigquery.py
##########
@@ -1915,6 +1927,11 @@ class ReadFromBigQuery(PTransform):
that dataset, and will remove it once it is not needed. Job needs
access
to create and delete tables within the given dataset. Dataset name
should *not* start with the reserved prefix `beam_temp_dataset_`.
+ query_priority (BigQueryQueryPriority): By default, this transform runs
Review comment:
never mind, my comment. I was thinking initially that the user doesn't
need to be aware of this parameter. There's no point executing the query if
there's not enough resources. Thus, batch priority for dataflow job makes
better sense.
--
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]
