[GitHub] [beam] tysonjh commented on a change in pull request #13170: [BEAM-9650] Adding support for ReadAll from BigQuery transform

2020-11-18 Thread GitBox
tysonjh commented on a change in pull request #13170: URL: https://github.com/apache/beam/pull/13170#discussion_r526495133 ## File path: sdks/python/apache_beam/io/gcp/bigquery_read_it_test.py ## @@ -298,6 +298,108 @@ def test_iobase_source(self): assert_that(result,

[GitHub] [beam] tysonjh commented on a change in pull request #13170: [BEAM-9650] Adding support for ReadAll from BigQuery transform

2020-11-13 Thread GitBox
tysonjh commented on a change in pull request #13170: URL: https://github.com/apache/beam/pull/13170#discussion_r523292482 ## File path: sdks/python/apache_beam/io/gcp/bigquery.py ## @@ -79,6 +79,41 @@ `ReadFromBigQuery`, you can use the flag `use_json_exports` to export

[GitHub] [beam] tysonjh commented on a change in pull request #13170: [BEAM-9650] Adding support for ReadAll from BigQuery transform

2020-11-11 Thread GitBox
tysonjh commented on a change in pull request #13170: URL: https://github.com/apache/beam/pull/13170#discussion_r520185234 ## File path: sdks/python/apache_beam/io/gcp/bigquery.py ## @@ -1919,3 +1890,77 @@ def file_path_to_remove(unused_elm): *self._args,