Skip to content

Migrate BigQuery tests to use BaseConnectorTest#7722

Closed
xiangzK wants to merge 6 commits intotrinodb:masterfrom
xiangzK:master
Closed

Migrate BigQuery tests to use BaseConnectorTest#7722
xiangzK wants to merge 6 commits intotrinodb:masterfrom
xiangzK:master

Conversation

@xiangzK
Copy link
Copy Markdown

@xiangzK xiangzK commented Apr 23, 2021

creating a pull request for Issue #7110

fixes #7110

@cla-bot
Copy link
Copy Markdown

cla-bot bot commented Apr 23, 2021

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please submit the signed CLA to cla@trino.io. For more information, see https://github.com/trinodb/cla.

@cla-bot
Copy link
Copy Markdown

cla-bot bot commented Apr 23, 2021

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please submit the signed CLA to cla@trino.io. For more information, see https://github.com/trinodb/cla.

@xiangzK xiangzK closed this Apr 23, 2021
@xiangzK xiangzK reopened this Apr 23, 2021
@cla-bot
Copy link
Copy Markdown

cla-bot bot commented Apr 24, 2021

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please submit the signed CLA to cla@trino.io. For more information, see https://github.com/trinodb/cla.


import static java.util.Objects.requireNonNull;

public class BigQueryReadRows
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Where do we use BigQueryReadRows ?

@ebyhr
Copy link
Copy Markdown
Member

ebyhr commented Mar 2, 2022

Closing as #11108

@ebyhr ebyhr closed this Mar 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

Migrate BigQuery tests to use BaseConnectorTest

4 participants