-
Notifications
You must be signed in to change notification settings - Fork 121
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Pandas GBQ Integration Tests #309
Comments
It appears to be How often are these errors showing up? |
Thanks @tswast Much more frequently than normal - several examples today. Recent example: |
I did some digging. googleapis/google-cloud-python#10219 added a default timeout to all requests. It seems the 60 seconds default was not high enough. Talking with some teammates at Google to see what might be causing this and what a more realistic timeout should be. |
Aah thanks very much for the investigation @tswast - much appreciated! Let us know if there is anything we should do on our side. |
@plamut believes this should be fixed in the latest version of the |
I would try with |
How often can the flakiness be observed, and how easy/difficult is to reproduce it locally? Also, would it be feasible to set up a CI check that would run the tests with |
Marking as a duplicate of #418 |
Pandas CI -> test_gbq.py can be flakey.
Following API call times out:
Called from https://github.com/pydata/pandas-gbq/blob/master/pandas_gbq/gbq.py#L1312
Can we allow
to_gbq
to set a more generous timeout? Or any other solution here?Example test failures on our master branch:
e.g : https://travis-ci.org/pandas-dev/pandas/jobs/646554600?utm_medium=notification&utm_source=github_status
and
https://travis-ci.org/pandas-dev/pandas/jobs/646597032?utm_medium=notification&utm_source=github_status
cc @jreback @tswast
The text was updated successfully, but these errors were encountered: