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

Include field name in Parquet PrimitiveTypeBuilder error messages #2804

Closed
andygrove opened this issue Sep 29, 2022 · 0 comments · Fixed by #2805
Closed

Include field name in Parquet PrimitiveTypeBuilder error messages #2804

andygrove opened this issue Sep 29, 2022 · 0 comments · Fixed by #2805
Assignees
Labels
enhancement Any new improvement worthy of a entry in the changelog parquet Changes to the parquet crate

Comments

@andygrove
Copy link
Member

Is your feature request related to a problem or challenge? Please describe what you are trying to do.

I am getting this error when attempting to write a Parquet file:

Cannot represent FIXED_LEN_BYTE_ARRAY as DECIMAL with length 3 and precision 7.

It would be convenient if the error included the name of the field causing this error. The field name is available at this point in the code.

Describe the solution you'd like

Add field name to error

Describe alternatives you've considered

None

Additional context

@andygrove andygrove added the enhancement Any new improvement worthy of a entry in the changelog label Sep 29, 2022
@andygrove andygrove self-assigned this Sep 29, 2022
@alamb alamb added the parquet Changes to the parquet crate label Sep 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Any new improvement worthy of a entry in the changelog parquet Changes to the parquet crate
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants