Skip to content

bigquery: reduce test flakiness in testCancelJob integration test. - #5592

Merged
shollyman merged 1 commit into
googleapis:masterfrom
shollyman:testflake
Jun 28, 2019
Merged

bigquery: reduce test flakiness in testCancelJob integration test.#5592
shollyman merged 1 commit into
googleapis:masterfrom
shollyman:testflake

Conversation

@shollyman

Copy link
Copy Markdown
Contributor

BigQuery's cancellation mechanism is best-effort notification to
the job server running cancellation. Depending on a timing factors,
the cancellation can be ignored, arrive after job completion, or
arrive in time to trigger premature state transition (which will yield an error).

This change removes the expectation that cancelling a job won't yield an error,
which is incorrect for the last case (which is triggered rarely).

BigQuery's cancellation mechanism is best-effort notification to
the job server running cancellation. Depending on a timing factors,
the cancellation can be ignored, arrive after job completion, or
arrive in time to trigger state transition (which will yield an error).
This change removes the expectation that cancelling a job won't
yield an error, which is incorrect for the last case.
@shollyman
shollyman requested a review from a teamJune 28, 2019 20:46
@googlebotgooglebot added the cla: yes This human has signed the Contributor License Agreement. label Jun 28, 2019
@codecov

codecovBot commented Jun 28, 2019

Copy link
Copy Markdown

Codecov Report

Merging #5592 into master will increase coverage by <.01%.
The diff coverage is n/a.

Impacted file tree graph

@@ Coverage Diff @@## master #5592 +/- ##
============================================
+ Coverage 46.71% 46.71% +<.01% 
Complexity 24624 24624 ============================================
Files 2351 2351 Lines 256121 256121 Branches 29323 29323 ============================================
+ Hits 119639 119645 +6 + Misses 127563 127557 -6 
Partials 8919 8919
Impacted FilesCoverage ΔComplexity Δ
.../cloud/datastore/testing/LocalDatastoreHelper.java85.07% <0%> (+4.47%)17% <0%> (ø)⬇️
.../java/com/google/cloud/testing/CommandWrapper.java96.96% <0%> (+9.09%)13% <0%> (ø)⬇️

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 b8eeb60...b132d49. Read the comment docs.

@shollyman
shollyman merged commit d0b16af into googleapis:masterJun 28, 2019
@shollyman
shollyman deleted the testflake branch June 28, 2019 22:29
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.

3 participants

@shollyman@tswast@googlebot