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.

Sign up for GitHub

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

Jump to bottom

Combine DecimalArray validation #2447

Closed
Tracked by #2313
liukun4515 opened this issue Aug 15, 2022 · 0 comments · Fixed by #2446
Closed
Tracked by #2313

Combine DecimalArray validation #2447

liukun4515 opened this issue Aug 15, 2022 · 0 comments · Fixed by #2446
Labels
arrow Changes to the arrow crate

Comments

@liukun4515
Copy link
Contributor

liukun4515 commented Aug 15, 2022 •
edited
Loading

Decimal array has two type: Decimal128Array and Decimal256Array.
The two type of decimal array have with_precision_scale respectively, we need to collation them into the common implementation.
This was referenced Aug 15, 2022
Closed
Merged
@alamb alamb changed the title collation validate precision for decimal array Combine decimal array validation Aug 17, 2022
@alamb alamb changed the title Combine decimal array validation Combine DecimalArray validation Aug 17, 2022
@alamb alamb added the arrow Changes to the arrow crate label Aug 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
arrow Changes to the arrow crate
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Collocate Decimal Array Validation Logic liukun4515/arrow-rs
2 participants
@alamb @liukun4515