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.

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

Fix: Add response object to any append rows requests exception #838

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Commits on Oct 11, 2024

  1. Fix: Add response object to any append rows requests exception

    Should an AppendRowsRequest fail, you need to inspect the response to
    see what went wrong. Currently this lib only raises an exception with
    the code and message, throwing the actual response away. This patch adds
    the response to any exception raise. This is fine because the base grpc
    error has a response kwarg that this lib wasn't using.
    
    Now you can catch the error and call `e.response.row_errors` to see the
    underlying row errors.
    
    Fixes: googleapis#836
    film42 committed Oct 11, 2024
    Configuration menu
    Copy the full SHA
    eeb9f50 View commit details
    Browse the repository at this point in the history

Commits on Oct 16, 2024

  1. Configuration menu
    Copy the full SHA
    62b1375 View commit details
    Browse the repository at this point in the history

Commits on Nov 15, 2024

  1. Configuration menu
    Copy the full SHA
    ca73906 View commit details
    Browse the repository at this point in the history