Skip to content
This repository was archived by the owner on Jun 8, 2026. It is now read-only.

feat(db_api): support JSON data type - #627

Merged
skuruppu merged 13 commits into
googleapis:mainfrom
MaxxleLLC:json_type
Nov 22, 2021
Merged

feat(db_api): support JSON data type#627
skuruppu merged 13 commits into
googleapis:mainfrom
MaxxleLLC:json_type

Conversation

@IlyaFaer

Copy link
Copy Markdown

No description provided.

@google-clagoogle-claBot added the cla: yes This human has signed the Contributor License Agreement. label Oct 18, 2021
@product-auto-labelproduct-auto-labelBot added the api: spanner Issues related to the googleapis/python-spanner API. label Oct 18, 2021
@IlyaFaer
IlyaFaer marked this pull request as ready for review October 18, 2021 10:29
@IlyaFaer
IlyaFaer requested a review from a teamOctober 18, 2021 10:29
@IlyaFaer
IlyaFaer requested a review from a teamOctober 25, 2021 10:54
@larkeelarkee added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Oct 28, 2021
@yoshi-kokoroyoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Oct 28, 2021
@larkeelarkee added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Oct 29, 2021
@yoshi-kokoroyoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Oct 29, 2021
@larkeelarkee added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Oct 30, 2021
@yoshi-kokoroyoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Oct 30, 2021
@larkeelarkee added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Oct 31, 2021
@yoshi-kokoroyoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Oct 31, 2021
@larkee

Copy link
Copy Markdown
Contributor

@IlyaFaer There are system test and sample failures relating to JSON comparisons because the assertions are expecting a JSON string but are getting a Python dict. This can be fixed by overriding the __repr__ method on JsonObject to return self.serialize().

@IlyaFaer
IlyaFaer requested a review from a team as a code ownerNovember 1, 2021 09:18
@IlyaFaer
IlyaFaer requested a review from nicainNovember 1, 2021 09:18
@larkeelarkee changed the title feat: support JSON data typefeat(dbapi): support JSON data typeNov 1, 2021
@larkeelarkee changed the title feat(dbapi): support JSON data typefeat(db_api): support JSON data typeNov 1, 2021
@IlyaFaerIlyaFaer added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Nov 2, 2021
@yoshi-kokoroyoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Nov 2, 2021
@larkee

Copy link
Copy Markdown
Contributor

@IlyaFaer There are lint errors and the sample assertions are still failing.

@IlyaFaer

Copy link
Copy Markdown
Author

@larkee, oh, looks like some samples requires some work. Working on it...

@IlyaFaer

Copy link
Copy Markdown
Author

@larkee, this one seems to be okay for merge.

@skuruppu
skuruppu merged commit d760c2c into googleapis:mainNov 22, 2021
@IlyaFaer
IlyaFaer deleted the json_type branch November 23, 2021 09:09
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: spannerIssues related to the googleapis/python-spanner API.cla: yesThis human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@IlyaFaer@larkee@skuruppu@yoshi-kokoro