Skip to content

feat(spanner): allow string values for Scan functions #11898

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

Merged
merged 4 commits into from
Mar 26, 2025

Conversation

olavloite
Copy link
Contributor

Support scanning string values for the various
spanner.NullTypes.

Use the Scan implementation of the underlying data type for spanner.NullDate, as civil.Date now implements the sql.Scanner interface.

The Scan implementations are used by the database/sql driver and applications that depend on the database/sql driver.

Support scanning string values for the various
spanner.NullTypes.

Use the Scan implementation of the underlying data type for
spanner.NullDate, as civil.Date now implements the sql.Scanner
interface.

The Scan implementations are used by the database/sql driver
and applications that depend on the database/sql driver.
@olavloite olavloite requested a review from rahul2393 March 24, 2025 17:42
@olavloite olavloite requested review from a team as code owners March 24, 2025 17:42
@product-auto-label product-auto-label bot added the api: spanner Issues related to the Spanner API. label Mar 24, 2025
@rahul2393 rahul2393 added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Mar 24, 2025
@kokoro-team kokoro-team removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Mar 24, 2025
@olavloite olavloite merged commit 9989dd0 into main Mar 26, 2025
9 checks passed
@olavloite olavloite deleted the spanner-scan-string-values branch March 26, 2025 12:46
@derekperkins
Copy link
Contributor

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 Spanner API.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants