satybald commented on a change in pull request #15536:
URL: https://github.com/apache/beam/pull/15536#discussion_r713282115
##########
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:
I am wondering can we make a batch priority by default in
ReadFromBigQuery? What would be the use case for the user to make it
interactive? I just want to make sure that the parameter that got exposed are
really necessary for the end user and it confirms with style guide
https://beam.apache.org/contribute/ptransform-style-guide/
--
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]
