Skip to content
This repository was archived by the owner on Apr 7, 2026. It is now read-only.

chore: refactor client side statements to accept the entire parsed statement#2556

Merged
olavloite merged 2 commits intomainfrom
parsed-statement-to-client-executors
Aug 2, 2023
Merged

chore: refactor client side statements to accept the entire parsed statement#2556
olavloite merged 2 commits intomainfrom
parsed-statement-to-client-executors

Conversation

@olavloite
Copy link
Copy Markdown
Collaborator

@olavloite olavloite commented Jul 28, 2023

Refactor the internal interface of client-side statements so these receive the entire parsed statement, including any query parameters in the statement. This allows us to create client-side statements that actually use the query parameters that have been specified by the user.

This refactor was pulled out of #2540 to make that PR smaller.

…atement

Refactor the internal interface of client-side statements so these receive the
entire parsed statement, including any query parameters in the statement. This
allows us to create client-side statements that actually use the query parameters
that have been specified by the user.
@olavloite olavloite requested a review from a team July 28, 2023 14:16
@product-auto-label product-auto-label Bot added size: m Pull request size is medium. api: spanner Issues related to the googleapis/java-spanner API. labels Jul 28, 2023
@olavloite olavloite merged commit c34d51e into main Aug 2, 2023
@olavloite olavloite deleted the parsed-statement-to-client-executors branch August 2, 2023 10:03
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants