feat(spanner): add support for Proto Columns (#6886)#2
Open
rahul2393 wants to merge 3 commits intohandling-null-values-in-proto-columnsfrom
Open
feat(spanner): add support for Proto Columns (#6886)#2rahul2393 wants to merge 3 commits intohandling-null-values-in-proto-columnsfrom
rahul2393 wants to merge 3 commits intohandling-null-values-in-proto-columnsfrom
Conversation
* feat(spanner): Adding support for Proto Message and Proto Enum * feat(spanner): Add license header to new files * code refactoring and additional checks * nit: code changes * go.mod update to prevent failing builds * go.mod update to prevent failing builds * go.mod and build_samples.sh update to prevent failing builds * revert back grpc version * revert back changes * Using standard Singers example, refactoring redundant code * code and proto refactoring * Add proto_type_fqn for Proto Message and Proto Enum * code refactoring * go mod tidy: go.mod and go.sum version updates * add changes to support compatibility between Int64 and Enum & compatibility between Bytes and Proto * Revert "go mod tidy: go.mod and go.sum version updates" This reverts commit 484b00c. * add Integration Tests for Proto Message, Proto Enum, compatibility tests * code refactoring * code refactoring * add unit tests for nil proto types * Add error when nil proto message or nil enum is passed
…oogleapis#6954) * feat(spanner): add support for handling null value in Proto columns * code refactor: get protoTypeFqn from user defined nil types * code refactoring * code refactoring * code refactoring * code refactoring * Add tests for MarshalJSON and UnmarshalJSON methods * refactoring test file * code refactoring
* feat(spanner): add support for array of proto columns * refactoring comments and added negative test cases while reading array of protos * change decoding logic of handling array of proto columns
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
feat(spanner): Adding support for Proto Message and Proto Enum
feat(spanner): Add license header to new files
code refactoring and additional checks
nit: code changes
go.mod update to prevent failing builds
go.mod update to prevent failing builds
go.mod and build_samples.sh update to prevent failing builds
revert back grpc version
revert back changes
Using standard Singers example, refactoring redundant code
code and proto refactoring
Add proto_type_fqn for Proto Message and Proto Enum
code refactoring
go mod tidy: go.mod and go.sum version updates
add changes to support compatibility between Int64 and Enum & compatibility between Bytes and Proto
Revert "go mod tidy: go.mod and go.sum version updates"
This reverts commit 484b00c.
add Integration Tests for Proto Message, Proto Enum, compatibility tests
code refactoring
code refactoring
add unit tests for nil proto types
Add error when nil proto message or nil enum is passed