NVIDIA / NVIDIA/cudf

[FEA] Handle `binary decimal`s in the main decoder in Parquet reader

Open
#18,384 0 comments 0 reactions 0 assignees View on GitHub
cuIO feature request libcudf
Dominant language
C++
Stars
9.8k
Forks
1.1k
Avg merge
3d 6m
Merged PRs (30d)
278

Description

**Is your feature request related to a problem? Please describe.**
Currently, the catch-all decoder in Parquet reader is being used to decode the `binary decimal`s, which are not handled by one of our main decoder. Therefore, we should handle `binary decimal`s in our main decoder(s) so that we can move one step closer to purging the catch-all decoder from the codebase.

Related to #18347

**Describe the solution you'd like**
`binary decimal` types being decoded in our main plain decoder instead of in the backup catch-all decoder.

**Describe alternatives you've considered**
N/A

**Additional context**
Here is the parquet file containing binary decimals: [binary_decimal.parquet.zip](https://github.com/user-attachments/files/19477476/binary_decimal.parquet.zip). This file has been generated by the cudf pytest named `test_parquet_reader_binary_decimal`

CC: @GregoryKimball @pmattione-nvidia @nvdbaranec

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.