Skip to content
This repository was archived by the owner on Mar 23, 2026. It is now read-only.

fix: make sure to fall back to old query path when query job is incomplete#941

Merged
stephaniewang526 merged 5 commits intogoogleapis:masterfrom
stephaniewang526:fix-bug
Nov 13, 2020
Merged

fix: make sure to fall back to old query path when query job is incomplete#941
stephaniewang526 merged 5 commits intogoogleapis:masterfrom
stephaniewang526:fix-bug

Conversation

@stephaniewang526
Copy link
Copy Markdown
Contributor

@stephaniewang526 stephaniewang526 commented Nov 12, 2020

When the query job takes more than 10s we must fall back to the old query path!

cc @epavan

@stephaniewang526 stephaniewang526 requested review from a team and shollyman November 12, 2020 19:58
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Nov 12, 2020
@product-auto-label product-auto-label bot added the api: bigquery Issues related to the googleapis/java-bigquery API. label Nov 12, 2020
@codecov
Copy link
Copy Markdown

codecov bot commented Nov 12, 2020

Codecov Report

Merging #941 (5aa0f7f) into master (22e8bd7) will increase coverage by 0.07%.
The diff coverage is 80.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #941      +/-   ##
============================================
+ Coverage     80.33%   80.40%   +0.07%     
- Complexity     1270     1271       +1     
============================================
  Files            79       79              
  Lines          6574     6573       -1     
  Branches        759      759              
============================================
+ Hits           5281     5285       +4     
+ Misses          897      893       -4     
+ Partials        396      395       -1     
Impacted Files Coverage Δ Complexity Δ
...n/java/com/google/cloud/bigquery/BigQueryImpl.java 83.08% <80.00%> (+0.71%) 86.00 <0.00> (+1.00)

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 22e8bd7...5aa0f7f. Read the comment docs.

@stephaniewang526 stephaniewang526 merged commit bd7d85c into googleapis:master Nov 13, 2020
gcf-merge-on-green bot pushed a commit that referenced this pull request Nov 13, 2020
🤖 I have created a release \*beep\* \*boop\* 
---
### [1.124.6](https://www.github.com/googleapis/java-bigquery/compare/v1.124.5...v1.124.6) (2020-11-13)


### Bug Fixes

* make sure to fall back to old query path when query job is incomplete ([#941](https://www.github.com/googleapis/java-bigquery/issues/941)) ([bd7d85c](https://www.github.com/googleapis/java-bigquery/commit/bd7d85c489cd260feeabbdc9ecbb8dcdc8d9ae77))


### Dependencies

* update dependency com.google.cloud:google-cloud-bigtable to v1.17.3 ([#936](https://www.github.com/googleapis/java-bigquery/issues/936)) ([0b34a50](https://www.github.com/googleapis/java-bigquery/commit/0b34a5095b2bad4ac02b3d40937dd1a3ab3f2154))
---


This PR was generated with [Release Please](https://github.com/googleapis/release-please).
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: bigquery Issues related to the googleapis/java-bigquery API. cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants