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

error previewing parquet file: invalid schema for type: DECIMAL, can not handle precision over 18 #7240

Closed
3 tasks done
jasonhorner opened this issue Sep 1, 2023 · 2 comments
Labels
✅ duplicate Issue or pull request already exists

Comments

@jasonhorner
Copy link

Preflight Checklist

Storage Explorer Version

1.31.1

Regression From

No response

Architecture

x64

Storage Explorer Build Number

20230822.1

Platform

All

OS Version

No response

Bug Description

when trying to preview a parquet. I encounter this error:

{
"name": null,
"message": ""invalid schema for type: DECIMAL, for Column: MyColumn, can not handle precision over 18""
}

the column MyColumn data type is money in the underlying sql server database. the parquet file was generated from the ADF Copy activity the parquet file seems valid.

As a side note while there is an error shown in the activities tab. the preview u/i shows the message "No Data" this is potentially misleading and I feel like it should say something like "Error Encountered" as no data implies at least to me that the file was empty.

here is a screenshot for context:
image

Steps to Reproduce

See Above

Actual Experience

see Above

Expected Experience

No response

Additional Context

No response

@JasonYeMSFT
Copy link
Contributor

JasonYeMSFT commented Sep 1, 2023

This is currently not supported. We will semi-support it in 1.32.0 by unblocking parsing when there are decimal fields whose underlying representation cannot be easily parsed (e.g. fixed length byte array, non-fixed length byte array) whose precision may be beyond 18.
#7042

@jasonhorner
Copy link
Author

@JasonYeMSFT not sure how I missed this issue last night... I did look. I do think the error messaging visibility in the preview window could be improved. Hope you will consider this for a future build. the preview features especially for parquet and Avro have been very useful.

@MRayermannMSFT MRayermannMSFT added the ✅ duplicate Issue or pull request already exists label Sep 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✅ duplicate Issue or pull request already exists
Projects
None yet
Development

No branches or pull requests

3 participants