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

chore(spanner): add precommit token support for mux R/W #3341

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Commits on Sep 19, 2024

  1. test: ignore if the mock server failed to return a row (#3335)

    The test uses a trick in the mock server, where the mock server is
    requested to freeze after returning the first row. However, when the
    mock server adds the first row to the stream, it is not guaranteed to be
    readable for the client, which again causes the test to hang on the
    ResultSet#next() call. The gRPC libraries then execute keep-alive
    requests to keep the TCP connection alive while waiting for data from
    the mock server, which will never come. This caused the query to
    eventually fail with a RESOURCE_EXHAUSTED error.
    
    The tests work around this issue by just ignoring the case when the mock
    server fails to return the first row, as it is something that only very
    sporadically happens.
    olavloite authored and rahul2393 committed Sep 19, 2024
    Configuration menu
    Copy the full SHA
    fae1585 View commit details
    Browse the repository at this point in the history

Commits on Sep 20, 2024

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