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

[FEA] Add support for unsigned types in the Parquet reader/writer #5352

Closed
davidwendt opened this issue Jun 1, 2020 · 0 comments · Fixed by #5528
Closed

[FEA] Add support for unsigned types in the Parquet reader/writer #5352

davidwendt opened this issue Jun 1, 2020 · 0 comments · Fixed by #5528
Assignees
@devavret
Labels
cuIO cuIO issue feature request New feature or request libcudf Affects libcudf (C++/CUDA) code.

Comments

@davidwendt
Copy link
Contributor

#5292 will add support for unsigned types in libcudf columns.
The Parquet reader and writers code will need to be updated to support these new types: UINT8, UINT16, UINT32, and UINT64.

https://github.com/apache/parquet-format/blob/master/LogicalTypes.md#numeric-types
@davidwendt davidwendt added feature request New feature or request Needs Triage Need team to review and classify labels Jun 1, 2020
@kkraus14 kkraus14 added libcudf Affects libcudf (C++/CUDA) code. cuIO cuIO issue and removed Needs Triage Need team to review and classify labels Jun 5, 2020
@rgsl888prabhu rgsl888prabhu mentioned this issue Jun 11, 2020
Merged
@kkraus14 kkraus14 mentioned this issue Jun 15, 2020
Closed
@devavret devavret self-assigned this Jun 18, 2020
@devavret devavret mentioned this issue Jun 22, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@devavret devavret

Labels
cuIO cuIO issue feature request New feature or request libcudf Affects libcudf (C++/CUDA) code.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[REVIEW] Unsigned int support for parquet reader and writer devavret/cudf
3 participants
@devavret @kkraus14 @davidwendt