Skip to content
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.

Sign up for GitHub

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

Jump to bottom

feat: retry failed query jobs in result() #837

Merged
merged 32 commits into from
Aug 11, 2021
Merged

feat: retry failed query jobs in result() #837

merged 32 commits into from
Aug 11, 2021

Commits on Jul 28, 2021

  1. initial stab

    @jimfulton
    jimfulton committed Jul 28, 2021
    Configuration menu
    Copy the full SHA
    cc62448 View commit details
    Browse the repository at this point in the history

  2. test start

    @jimfulton
    jimfulton committed Jul 28, 2021
    Configuration menu
    Copy the full SHA
    61e1c38 View commit details
    Browse the repository at this point in the history

  3. Test high-level retry behavior.

    @jimfulton
    jimfulton committed Jul 28, 2021
    Configuration menu
    Copy the full SHA
    093f9b4 View commit details
    Browse the repository at this point in the history

  4. Don't retry here 

    Retry in the quert job's result method.
    @jimfulton
    jimfulton committed Jul 28, 2021
    Configuration menu
    Copy the full SHA
    b2ce74b View commit details
    Browse the repository at this point in the history

  5. reworked the retry logic.

    @jimfulton
    jimfulton committed Jul 28, 2021
    Configuration menu
    Copy the full SHA
    d930c83 View commit details
    Browse the repository at this point in the history

Commits on Jul 29, 2021

  1. if, when retrying, the new query job is complete and errorer, stop ri… 

    …ght away

There's no point in continuing to make calls.
    @jimfulton
    jimfulton committed Jul 29, 2021
    Configuration menu
    Copy the full SHA
    bf7b4c6 View commit details
    Browse the repository at this point in the history

  2. Added test (and to existing test) to make sure we can call result mul… 

    …tiple times
    @jimfulton
    jimfulton committed Jul 29, 2021
    Configuration menu
    Copy the full SHA
    0598197 View commit details
    Browse the repository at this point in the history

  3. Keep carrying retry_do_query, even though it shouldn't be necessary.

    @jimfulton
    jimfulton committed Jul 29, 2021
    Configuration menu
    Copy the full SHA
    c9644e9 View commit details
    Browse the repository at this point in the history

  4. blacken

    @jimfulton
    jimfulton committed Jul 29, 2021
    Configuration menu
    Copy the full SHA
    2d0e067 View commit details
    Browse the repository at this point in the history

  5. removed unecessary condition 

    that caused a coverage fail
    @jimfulton
    jimfulton committed Jul 29, 2021
    Configuration menu
    Copy the full SHA
    0dcac01 View commit details
    Browse the repository at this point in the history

  6. System test that demonstrates the retry behavior as applied to the or… 

    …iginal issues' use case
    @jimfulton
    jimfulton committed Jul 29, 2021
    Configuration menu
    Copy the full SHA
    026edba View commit details
    Browse the repository at this point in the history

  7. Added missing copyright

    @jimfulton
    jimfulton committed Jul 29, 2021
    Configuration menu
    Copy the full SHA
    0e764d2 View commit details
    Browse the repository at this point in the history

  8. Added missing copyright

    @jimfulton
    jimfulton committed Jul 29, 2021
    Configuration menu
    Copy the full SHA
    c96d8b3 View commit details
    Browse the repository at this point in the history

Commits on Aug 4, 2021

  1. Merge branch 'master' into fix-retry

    @tswast
    tswast authored Aug 4, 2021
    Configuration menu
    Copy the full SHA
    5058cb4 View commit details
    Browse the repository at this point in the history

  2. Added a leading _ to the retry_do_query query-jov attribute to make i… 

    …t clear that this is an implementation detail
    @jimfulton
    jimfulton committed Aug 4, 2021
    Configuration menu
    Copy the full SHA
    3c53172 View commit details
    Browse the repository at this point in the history

Commits on Aug 5, 2021

  1. Merge branch 'master' into fix-retry

    @tswast
    tswast authored Aug 5, 2021
    Configuration menu
    Copy the full SHA
    eb51432 View commit details
    Browse the repository at this point in the history

  2. fixed copyright

    @jimfulton
    jimfulton committed Aug 5, 2021
    Configuration menu
    Copy the full SHA
    d6d958f View commit details
    Browse the repository at this point in the history

  3. Merge branch 'fix-retry' of github.com:jimfulton/python-bigquery into… 

    … fix-retry
    @jimfulton
    jimfulton committed Aug 5, 2021
    Configuration menu
    Copy the full SHA
    a05f75f View commit details
    Browse the repository at this point in the history

  4. why sleep?

    @jimfulton
    jimfulton committed Aug 5, 2021
    Configuration menu
    Copy the full SHA
    32e7050 View commit details
    Browse the repository at this point in the history

  5. use DEFAULT_RETRY for low-level requests, to retry API errors

    @jimfulton
    jimfulton committed Aug 5, 2021
    Configuration menu
    Copy the full SHA
    3361a82 View commit details
    Browse the repository at this point in the history

Commits on Aug 9, 2021

  1. Separate job retry into separate option for client query and query-jo… 

    …b result
    @jimfulton
    jimfulton committed Aug 9, 2021
    Configuration menu
    Copy the full SHA
    4c6ef5b View commit details
    Browse the repository at this point in the history

  2. Merge branch 'master' into fix-retry

    @jimfulton
    jimfulton authored Aug 9, 2021
    Configuration menu
    Copy the full SHA
    25b44a0 View commit details
    Browse the repository at this point in the history

Commits on Aug 10, 2021

  1. Use None job_retry to disable retry

    @jimfulton
    jimfulton committed Aug 10, 2021
    Configuration menu
    Copy the full SHA
    9ab84e4 View commit details
    Browse the repository at this point in the history

  2. Use a 10-minute deadline for job retry by default

    @jimfulton
    jimfulton committed Aug 10, 2021
    Configuration menu
    Copy the full SHA
    d2bf840 View commit details
    Browse the repository at this point in the history

  3. Merge branch 'fix-retry' of github.com:jimfulton/python-bigquery into… 

    … fix-retry
    @jimfulton
    jimfulton committed Aug 10, 2021
    Configuration menu
    Copy the full SHA
    4c004a5 View commit details
    Browse the repository at this point in the history

  4. Merge branch 'master' into fix-retry

    @jimfulton
    jimfulton authored Aug 10, 2021
    Configuration menu
    Copy the full SHA
    be49c4b View commit details
    Browse the repository at this point in the history

Commits on Aug 11, 2021

  1. Merge branch 'master' into fix-retry

    @jimfulton
    jimfulton authored Aug 11, 2021
    Configuration menu
    Copy the full SHA
    9e4a011 View commit details
    Browse the repository at this point in the history

  2. Merge branch 'master' into fix-retry

    @jimfulton
    jimfulton authored Aug 11, 2021
    Configuration menu
    Copy the full SHA
    204641b View commit details
    Browse the repository at this point in the history

  3. Added another default reason to retry jobs

    @jimfulton
    jimfulton committed Aug 11, 2021
    Configuration menu
    Copy the full SHA
    bf051b0 View commit details
    Browse the repository at this point in the history

  4. Merge branch 'fix-retry' of github.com:jimfulton/python-bigquery into… 

    … fix-retry
    @jimfulton
    jimfulton committed Aug 11, 2021
    Configuration menu
    Copy the full SHA
    b47244f View commit details
    Browse the repository at this point in the history

  5. Merge branch 'master' into fix-retry

    @jimfulton
    jimfulton authored Aug 11, 2021
    Configuration menu
    Copy the full SHA
    2377a1b View commit details
    Browse the repository at this point in the history

  6. Update tests/unit/test_job_retry.py

    @tswast
    tswast authored Aug 11, 2021
    Configuration menu
    Copy the full SHA
    6b790e8 View commit details
    Browse the repository at this point in the history