tvalentyn commented on a change in pull request #11086:
URL: https://github.com/apache/beam/pull/11086#discussion_r418609817
##########
File path: sdks/python/apache_beam/io/gcp/bigquery.py
##########
@@ -45,8 +45,8 @@
may use some caching techniques to share the side inputs between calls in order
to avoid excessive reading:::
- main_table = pipeline | 'VeryBig' >> beam.io.Read(beam.io.BigQuerySource()
- side_table = pipeline | 'NotBig' >> beam.io.Read(beam.io.BigQuerySource()
+ main_table = pipeline | 'VeryBig' >> beam.io.ReadFroBigQuery(...)
Review comment:
Typo in From
##########
File path: sdks/python/apache_beam/io/gcp/bigquery.py
##########
@@ -78,6 +72,12 @@
or a table. Pipeline construction will fail with a validation error if neither
or both are specified.
+When reading from BigQuery using `BigQuerySource`, bytes are returned as
Review comment:
Can we add some guidance when to use which?
----------------------------------------------------------------
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]
