Skip to content

Add initial integration test for BigQuery Storage API. - #5216

Merged
tswast merged 1 commit into
googleapis:masterfrom
mmladenovski:bq_storage_it_test
May 23, 2019
Merged

Add initial integration test for BigQuery Storage API.#5216
tswast merged 1 commit into
googleapis:masterfrom
mmladenovski:bq_storage_it_test

Conversation

@mmladenovski

Copy link
Copy Markdown

Integration tests in this change cover:

  • simple/non-filtered read from a table
  • filtered read from a table
  • read that only selects certain columns

In addition to adding integration tests, this change also includes the Kokoro setup for presubmit, nighthly and continuous builds.

In upcoming PRs for integration tests, other aspects of the BigQuery Storage API will be covered.

@tswast, please review the newly added tests.
@chingor13 , please review the Kokoro setup.

CC: @shollyman , @kmjung , @aryann

Integration tests in this change cover:
- simple/non-filtered read from a table
- filtered read from a table
- read that only selects certain columns
In addition to adding integration tests, this change also includes the Kokoro setup for presubmit, nighthly and continuous builds.
@mmladenovski
mmladenovski requested a review from a teamMay 23, 2019 20:16
@googlebotgooglebot added the cla: yes This human has signed the Contributor License Agreement. label May 23, 2019
@chingor13
chingor13 requested review from chingor13 and tswast and removed request for a teamMay 23, 2019 20:19
@chingor13chingor13 added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label May 23, 2019
@yoshi-kokoroyoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label May 23, 2019

@chingor13chingor13 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Kokoro config LGTM. And it passed first time!

@codecov

codecovBot commented May 23, 2019

Copy link
Copy Markdown

Codecov Report

Merging #5216 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@ Coverage Diff @@## master #5216 +/- ##
=========================================
Coverage 50.41% 50.41% - Complexity 23796 23811 +15 
=========================================
Files 2251 2251 Lines 226836 226836 Branches 24966 24961 -5 =========================================
Hits 114370 114370 Misses 103858 103858 Partials 8608 8608

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7ef8b85...3067e6c. Read the comment docs.

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yesThis human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@mmladenovski@chingor13@tswast@googlebot@yoshi-kokoro