-
Notifications
You must be signed in to change notification settings - Fork 25
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
docs: add sample for all data types #51
Conversation
examples/data-types/main.go
Outdated
float64 sql.NullFloat64 | ||
numeric spanner.NullNumeric // There is no sql.NullNumeric type | ||
date spanner.NullDate // There is no sql.NullDate type | ||
timestamp spanner.NullTime |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
there is no sql.NullTime?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, there is. Good catch, thanks.
examples/go.mod
Outdated
@@ -5,13 +5,14 @@ go 1.14 | |||
replace github.com/cloudspannerecosystem/go-sql-spanner => ../ | |||
|
|||
require ( | |||
cloud.google.com/go/spanner v1.25.0 | |||
cloud.google.com/go v0.97.0 | |||
cloud.google.com/go/spanner v1.26.1-0.20211019074412-4537b45d2611 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do you want to upgrade to 1.27?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, thanks for releasing that so quickly.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
Adds a sample for how to use all supported data types.
This sample depends on very recent changes in the Spanner client library that have not yet been officially released: googleapis/google-cloud-go#4936