Skip to content

Warm up BQ service calls in IT tests#5708

Merged
clairemcginty merged 1 commit intomainfrom
warmup-bq-service-call
Jun 16, 2025
Merged

Warm up BQ service calls in IT tests#5708
clairemcginty merged 1 commit intomainfrom
warmup-bq-service-call

Conversation

@clairemcginty
Copy link
Contributor

@clairemcginty clairemcginty commented Jun 16, 2025

Copy link
Contributor

@charles-tan charles-tan left a comment

Choose a reason for hiding this comment

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

lgtm

val sqlQuery =
"SELECT word, word_count FROM `bigquery-public-data.samples.shakespeare` LIMIT 10"

// First BQ service call fails on GHA; see https://github.com/spotify/scio/issues/5702
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe add a comment to #5702 that it seems to be more of a github actions issue and not the code itself

@codecov
Copy link

codecov bot commented Jun 16, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 61.47%. Comparing base (be840f8) to head (8cf7632).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #5708   +/-   ##
=======================================
  Coverage   61.46%   61.47%           
=======================================
  Files         314      314           
  Lines       11324    11324           
  Branches      801      801           
=======================================
+ Hits         6960     6961    +1     
+ Misses       4364     4363    -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@clairemcginty clairemcginty merged commit 344e003 into main Jun 16, 2025
12 of 14 checks passed
@clairemcginty clairemcginty deleted the warmup-bq-service-call branch June 16, 2025 21:50
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.

2 participants