Skip to content

Handle BigQuery 503 error#1932

Closed
krmettu wants to merge 1 commit intoapache:masterfrom
krmettu:master
Closed

Handle BigQuery 503 error#1932
krmettu wants to merge 1 commit intoapache:masterfrom
krmettu:master

Conversation

@krmettu
Copy link

@krmettu krmettu commented Dec 8, 2016

Dear Airflow Maintainers,

Please accept this PR that addresses the following issues:

https://issues.apache.org/jira/browse/AIRFLOW-667

We have enoutered this issue few times in our production environment. I added this fix based on google's documentation.

https://cloud.google.com/bigquery/troubleshooting-errors

"This error returns when there is a temporary server failure such as a network connection problem or a server overload. In general, wait a few seconds and try again."

Testing Done:

  • Tests are done connecting to BQ tables using the PR code.

Subject - Handle BigQuery 503 error
added retry feature if BigQuery error is 500 or 503
set the retry time for 5 seconds

added retry feature if BigQuery error is 500 or 503
set the retry time for 5 seconds
@codecov-io
Copy link

codecov-io commented Dec 8, 2016

Current coverage is 66.55% (diff: 100%)

Merging #1932 into master will decrease coverage by <.01%

@@             master      #1932   diff @@
==========================================
  Files           135        135          
  Lines         10133      10133          
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
- Hits           6745       6744     -1   
- Misses         3388       3389     +1   
  Partials          0          0          

Powered by Codecov. Last update b674656...7dec320

@krmettu krmettu closed this Dec 9, 2016
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