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

fix(bigquerystorage): resume reader connection on EOS internal error #9994

Merged
merged 4 commits into from
Jan 15, 2020
Merged

fix(bigquerystorage): resume reader connection on EOS internal error #9994

merged 4 commits into from
Jan 15, 2020

Commits on Jan 8, 2020

  1. fix(bigquerystorage): resume reader connection on EOS internal error 

    It's infeasible for the backend to change the status of `EOS on DATA`
internal errors, so instead we check the error message to see if it's
an error that is resumable. We don't want to try to resume on *all*
internal errors, so inspecting the message is the best we can do.
    @tswast
    tswast committed Jan 8, 2020
    Configuration menu
    Copy the full SHA
    72f84a1 View commit details
    Browse the repository at this point in the history

  2. fix: update resumable errors to match string from test failure

    @tswast
    tswast committed Jan 8, 2020
    Configuration menu
    Copy the full SHA
    50ecc98 View commit details
    Browse the repository at this point in the history

Commits on Jan 15, 2020

  1. Merge remote-tracking branch 'upstream/master' into b143292803-eos-bq… 

    …storage
    @tswast
    tswast committed Jan 15, 2020
    Configuration menu
    Copy the full SHA
    1a4b881 View commit details
    Browse the repository at this point in the history

  2. refactor: use more readable any instead of loop

    @tswast
    tswast committed Jan 15, 2020
    Configuration menu
    Copy the full SHA
    642f714 View commit details
    Browse the repository at this point in the history