Skip to content

[BEAM-6621] Basic Bigquery IO tests on Python 3 PostCommit#7782

Merged
aaltay merged 1 commit into
apache:masterfrom
markflyhigh:py3-bq-it
Feb 8, 2019
Merged

[BEAM-6621] Basic Bigquery IO tests on Python 3 PostCommit#7782
aaltay merged 1 commit into
apache:masterfrom
markflyhigh:py3-bq-it

Conversation

@markflyhigh
Copy link
Copy Markdown
Contributor

@markflyhigh markflyhigh commented Feb 7, 2019

This change add bigquery_tornadoes_it_test to several Python 3 Jenkins builds:

  • Runs on DirectRunner in PreCommit and PostCommit (beam_PostCommit_Python_Verify and beam_PostCommit_Python_Verify_cron)
  • Runs on DataflowRunner in PostCommit (beam_PostCommit_Python_Verify)

+R: @tvalentyn @charlesccychen
+cc: @RobbeSneyders


Follow this checklist to help us incorporate your contribution quickly and easily:

  • Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
  • If this contribution is large, please file an Apache Individual Contributor License Agreement.

It will help us expedite review of your Pull Request if you tag someone (e.g. @username) to look at it.

Post-Commit Tests Status (on master branch)

Lang SDK Apex Dataflow Flink Gearpump Samza Spark
Go Build Status --- --- --- --- --- ---
Java Build Status Build Status Build Status Build Status
Build Status
Build Status
Build Status Build Status Build Status
Python Build Status --- Build Status
Build Status
Build Status --- --- ---

@markflyhigh
Copy link
Copy Markdown
Contributor Author

Run Python PostCommit

@markflyhigh
Copy link
Copy Markdown
Contributor Author

@tvalentyn @aaltay @charlesccychen PTAL

@tvalentyn
Copy link
Copy Markdown
Contributor

LGTM, thanks, @markflyhigh !

@aaltay aaltay merged commit 381ab55 into apache:master Feb 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants