Skip to content
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

feat(spanner): support NUMERIC in mutations #3328

Merged
merged 5 commits into from Dec 10, 2020

Conversation

@apstndb
Copy link
Contributor

@apstndb apstndb commented Dec 2, 2020

This PR adds support of NUMERIC type in mutations.
ref #3327

@apstndb apstndb requested review from skuruppu and as code owners Dec 2, 2020
@google-cla google-cla bot added the cla: yes label Dec 2, 2020
@olavloite olavloite self-requested a review Dec 2, 2020
Copy link
Collaborator

@olavloite olavloite left a comment

Good catch, thanks for fixing this.

It would have been good to have a test case that would note this in case there are new data types added in the future, but apparently that is missing. We can add that through a separate PR.

Loading

@olavloite
Copy link
Collaborator

@olavloite olavloite commented Dec 3, 2020

@tbpg Do you have any idea why the kokoro run seems to be hanging indefinitely? I've tried to trigger the execution a number of times, but it never seems to finish (or report a status).

Loading

gcf-merge-on-green bot pushed a commit that referenced this issue Dec 4, 2020
Adds an additional test case for the fix in #3328 

Updates #3327
@tbpg tbpg added the kokoro:run label Dec 9, 2020
@tbpg
Copy link
Collaborator

@tbpg tbpg commented Dec 9, 2020

@olavloite, sorry, just saw this. I added the label to start Kokoro. Must have missed the initial webhook or something.

Loading

@olavloite olavloite merged commit fa90737 into googleapis:master Dec 10, 2020
3 checks passed
Loading
@olavloite
Copy link
Collaborator

@olavloite olavloite commented Dec 10, 2020

@olavloite, sorry, just saw this. I added the label to start Kokoro. Must have missed the initial webhook or something.

No problem, thanks for looking into it :-)

Loading

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

5 participants