Skip to content

Conversation

@hengfengli
Copy link
Contributor

No description provided.

@hengfengli hengfengli requested review from a team as code owners November 14, 2023 06:10
@product-auto-label product-auto-label bot added the api: spanner Issues related to the Spanner API. label Nov 14, 2023
rahul2393
rahul2393 previously approved these changes Nov 14, 2023
@hengfengli hengfengli force-pushed the spanner-pg-oid-support branch from 36d6ded to 13e77c5 Compare November 14, 2023 06:18
when :PG_JSONB
V1::Type.new(code: :JSON, type_annotation: :PG_JSONB)
V1::Type.new(code: :JSON, type_annotation: :PG_JSONB)
when :PG_OID
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we also add a unit test in
ruby-spanner/google-cloud-spanner/test/google/cloud/spanner/convert/grpc_type_for_field_test.rb

@bajajneha27
Copy link
Collaborator

Hi @hengfengli
Are we going to continue to work on this PR?

@SandeepTuniki
Copy link
Contributor

@hengfengli Friendly ping!

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.

5 participants