Skip to content

Conversation

@agasheesh
Copy link
Contributor

Adds java integration tests which test the cloud spanner functionality around commit timestamps that are written atomically as part of a transaction.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jan 16, 2020
@agasheesh agasheesh force-pushed the commit_timestamp_tests branch from 1cdc696 to ec3d74a Compare January 16, 2020 07:27
@agasheesh agasheesh requested a review from skuruppu January 16, 2020 07:29
@agasheesh agasheesh changed the title Add Commit Timestamp java integration tests for cloud spanner test: add Commit Timestamp java integration tests for cloud spanner Jan 16, 2020
@agasheesh agasheesh changed the title test: add Commit Timestamp java integration tests for cloud spanner [WIP] test: add Commit Timestamp java integration tests for cloud spanner Jan 16, 2020
@agasheesh agasheesh changed the title [WIP] test: add Commit Timestamp java integration tests for cloud spanner test: [WIP] add Commit Timestamp java integration tests for cloud spanner Jan 16, 2020
@agasheesh agasheesh force-pushed the commit_timestamp_tests branch from 618e334 to a586526 Compare January 16, 2020 08:06
@skuruppu skuruppu requested a review from olavloite January 16, 2020 08:44
@agasheesh agasheesh force-pushed the commit_timestamp_tests branch from cc3cd78 to 56cd23e Compare January 22, 2020 00:30
@skuruppu skuruppu merged commit 5a4fc74 into googleapis:master Jan 23, 2020
rajatbhatta pushed a commit to rajatbhatta/java-spanner that referenced this pull request Nov 17, 2022
…s#37)

* feat: allow using existing OAuth token for JDBC connection

Allow the user to specify an existing OAuth token to use for a JDBC connection,
instead of requiring the user to specify a credentials file or using the default
credentials of the environment.

Fixes googleapis#29

* tests: fix test cases

* fix: remove unused method

* fix: use default credentials when no file found

* fix: fix faulty merge
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants