Skip to content

chore: make DatabaseClient accessible in the Connection API#1847

Merged
gcf-merge-on-green[bot] merged 5 commits intomainfrom
dbclient-accessible-in-connection
Apr 28, 2022
Merged

chore: make DatabaseClient accessible in the Connection API#1847
gcf-merge-on-green[bot] merged 5 commits intomainfrom
dbclient-accessible-in-connection

Conversation

@olavloite
Copy link
Collaborator

Makes the underlying DatabaseClient accessible in the Connection API so this can be used by drivers that need direct access to it.

See https://github.com/GoogleCloudPlatform/pgadapter/blob/8f0d47785e04dc9db6ee275074777849f006d797/src/main/java/com/google/cloud/spanner/pgadapter/utils/MutationWriter.java#L439

@olavloite olavloite requested review from a team and ansh0l April 22, 2022 04:21
@product-auto-label product-auto-label bot added size: s Pull request size is small. api: spanner Issues related to the googleapis/java-spanner API. labels Apr 22, 2022
@olavloite olavloite requested a review from a team April 22, 2022 04:24
Copy link
Contributor

@ansh0l ansh0l left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@rajatbhatta rajatbhatta left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@olavloite olavloite added the automerge Merge the pull request once unit tests and other checks pass. label Apr 28, 2022
@gcf-merge-on-green gcf-merge-on-green bot merged commit ca85276 into main Apr 28, 2022
@gcf-merge-on-green gcf-merge-on-green bot deleted the dbclient-accessible-in-connection branch April 28, 2022 13:14
@gcf-merge-on-green gcf-merge-on-green bot removed the automerge Merge the pull request once unit tests and other checks pass. label Apr 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: spanner Issues related to the googleapis/java-spanner API. size: s Pull request size is small.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants