Fixing incorrect use of dataset_id in system tests.#1383
Merged
dhermes merged 1 commit intogoogleapis:masterfrom Jan 14, 2016
Merged
Fixing incorrect use of dataset_id in system tests.#1383dhermes merged 1 commit intogoogleapis:masterfrom
dhermes merged 1 commit intogoogleapis:masterfrom
Conversation
Causes by googleapis#1330 and googleapis#1369 crossing paths.
googlebot
added
the
cla: yes
Contributor
Author
|
@tseaver I'm merging this to keep the build from staying in a failed state. We can roll back after the fact if there is an issue. |
dhermes
added a commit
that referenced
this pull request
Jan 14, 2016
Fixing incorrect use of dataset_id in system tests.
This was referenced Jan 25, 2016
parthea
pushed a commit
that referenced
this pull request
Nov 24, 2025
parthea
pushed a commit
that referenced
this pull request
Nov 26, 2025
parthea
added a commit
that referenced
this pull request
Mar 2, 2026
Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com> Co-authored-by: Anthonios Partheniou <[email protected]>
parthea
pushed a commit
that referenced
this pull request
Mar 9, 2026
parthea
pushed a commit
that referenced
this pull request
Mar 9, 2026
* Update `SessionOptions` to support `GOOGLE_CLOUD_SPANNER_FORCE_DISABLE_MULTIPLEXED_SESSIONS` and add unit tests. Signed-off-by: Taylor Curran <[email protected]> * feat: Multiplexed sessions - Remove handling of `MethodNotImplemented` exception from `DatabaseSessionManager` and add unit tests. Signed-off-by: Taylor Curran <[email protected]> * feat: Multiplexed sessions - Update `Connection` to use multiplexed sessions, add unit tests. Signed-off-by: Taylor Curran <[email protected]> * cleanup: Rename `beforeNextRetry` to `before_next_retry`. Signed-off-by: Taylor Curran <[email protected]> * cleanup: Fix a few unrelated typos. Signed-off-by: Taylor Curran <[email protected]> * feat: Multiplexed sessions - Add ingest of precommit tokens to `_SnapshotBase` and update attributes and tests. Signed-off-by: Taylor Curran <[email protected]> * feat: Multiplexed sessions - Deprecate `StreamedResultSet._source` (redundant as transaction ID is set via `_restart_on_unavailable`) Signed-off-by: Taylor Curran <[email protected]> * feat: Multiplexed sessions - Move `_session_options` from `Database` to `Client` so that multiplexed are disabled for _all_ databases. Signed-off-by: Taylor Curran <[email protected]> * feat: Multiplexed sessions - Deprecate `SessionCheckout` and update `Database.run_in_transaction` to not use it. Signed-off-by: Taylor Curran <[email protected]> * feat: Multiplexed sessions - Deprecate `Database.session()` and minor cleanup. Signed-off-by: Taylor Curran <[email protected]> * feat: Multiplexed sessions - Update `BatchSnapshot` to use database session manager. Signed-off-by: Taylor Curran <[email protected]> * feat: Multiplexed sessions - Move `Batch` and `Transaction` attributes from class attributes to instance attributes. Signed-off-by: Taylor Curran <[email protected]> * feat: Multiplexed sessions - Update pools so they don't use deprecated `database.session()` Signed-off-by: Taylor Curran <[email protected]> * feat: Multiplexed sessions - Update session to remove class attributes, add TODOs, and make `Session._transaction` default to None. Plus add some `Optional` typing hints. Signed-off-by: Taylor Curran <[email protected]> * feat: Multiplexed sessions - Move begin transaction logic from `Snapshot` to `_SnapshotBase` and update unit tests. Signed-off-by: Taylor Curran <[email protected]> * feat: Multiplexed sessions - Remove begin transaction logic from `Transaction`, move to base class, update tests. Signed-off-by: Taylor Curran <[email protected]> * feat: Multiplexed sessions - Add logic for beginning mutations-only transactions. Signed-off-by: Taylor Curran <[email protected]> * feat: Multiplexed sessions - Cleanup and improve consistency of state checks, add `raises` documentation. Signed-off-by: Taylor Curran <[email protected]> * feat: Multiplexed sessions - Cleanup documentation for `Batch.commit`, some minor cleanup. Signed-off-by: Taylor Curran <[email protected]> * feat: Multiplexed sessions - Add logic for retrying commits if precommit token returned. Signed-off-by: Taylor Curran <[email protected]> * feat: Multiplexed sessions - Remove `GOOGLE_CLOUD_SPANNER_FORCE_DISABLE_MULTIPLEXED_SESSIONS` and update tests. Signed-off-by: Taylor Curran <[email protected]> * feat: Multiplexed sessions - Cleanup `TestDatabaseSessionManager` so that it doesn't depend on environment variable values. Signed-off-by: Taylor Curran <[email protected]> * feat: Multiplexed sessions - Add type hints for `SessionOptions` and `DatabaseSessionManager`. Signed-off-by: Taylor Curran <[email protected]> * feat: Multiplexed sessions - Fix `test_observability_options` Signed-off-by: Taylor Curran <[email protected]> * feat: Multiplexed sessions - Update `_builders` to use mock scoped credentials. Signed-off-by: Taylor Curran <[email protected]> * feat: Multiplexed sessions - Add helpers for mock scoped credentials for testing. Signed-off-by: Taylor Curran <[email protected]> * feat: Multiplexed sessions - Fix failing `test_batch_insert_then_read`. Signed-off-by: Taylor Curran <[email protected]> * feat: Multiplexed sessions - Fix failing `test_transaction_read_and_insert_then_rollback`. Signed-off-by: Taylor Curran <[email protected]> * feat: Multiplexed sessions - Add test helper for multiplexed env vars. Signed-off-by: Taylor Curran <[email protected]> * feat: Multiplexed sessions - Add unit tests for begin transaction base class, simplify `_SnapshotBase` tests, remove redundant tests. Signed-off-by: Taylor Curran <[email protected]> * feat: Multiplexed sessions - Attempt to fix `test_transaction_read_and_insert_then_rollback` and add `build_request_id` helper method, fix `test_snapshot` and `test_transaction` failures. Signed-off-by: Taylor Curran <[email protected]> * feat: Multiplexed sessions - Add test for log when new session created by maintenance thread. Signed-off-by: Taylor Curran <[email protected]> * feat: Multiplexed sessions - Add additional multiplexed unit tests for `_SnapshotBase`. Signed-off-by: Taylor Curran <[email protected]> * feat: Multiplexed sessions - Cleanup `Transaction` by extracting some constants for next step. Signed-off-by: Taylor Curran <[email protected]> * feat: Multiplexed sessions - Add additional `Transaction` tests for new multiplexed behaviour. Signed-off-by: Taylor Curran <[email protected]> * feat: Multiplexed sessions - Fix linter Signed-off-by: Taylor Curran <[email protected]> * feat: Multiplexed sessions - Remove unnecessary TODO Signed-off-by: Taylor Curran <[email protected]> * feat: Multiplexed sessions - Remove unnecessary constants. Signed-off-by: Taylor Curran <[email protected]> * feat: Multiplexed sessions - Remove support for disabling the use of multiplexed sessions due to runtime failures. Signed-off-by: Taylor Curran <[email protected]> * feat: Multiplexed sessions - Make deprecation comments a bit more clear. Signed-off-by: Taylor Curran <[email protected]> * feat: Multiplexed sessions - Add some more type hints. Signed-off-by: Taylor Curran <[email protected]> --------- Signed-off-by: Taylor Curran <[email protected]>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Caused by #1330 and #1369 crossing paths.