Closed bharadwajvr closed 3 weeks ago
Here is the summary of changes.
This comment is generated by snippet-bot.
If you find problems with this result, please file an issue at:
https://github.com/googleapis/repo-automation-bots/issues.
To update this comment, add snippet-bot:force-run
label or use the checkbox below:
🤖 I detect that the PR title and the commit message differ and there's only one commit. To use the PR title for the commit history, you can use Github's automerge feature with squashing, or use automerge
label. Good luck human!
-- conventional-commit-lint bot https://conventionalcommits.org/
The new graph_snippet.go models spanner/spanner_snippets/snippet.go. I have tested this manually, but I am not sure where test code for the code in snippet.go is written, and similarly how graph_snippet.go should be set up with tests. Looking for some guidance on this. Thanks!
Gentle ping
Before we continue with this review, please verify that these samples are needed.
See the list of existing samples here: https://cloud.google.com/docs/samples?language=golang&product=cloudspanner
These samples address a new feature of Spanner - the ability to create and query property graphs.
Add a few code snippets to illustrate how to use Spanner Graph functionality in Golang. These examples resemble other snippets for how various other Spanner features can be used.
Fixes #4312
Checklist
go test -v ./..
(see Testing)gofmt
(see Formatting)go vet
(see Formatting)