Skip to content
This repository was archived by the owner on May 17, 2024. It is now read-only.

keep precision, scale info for databricks decimal #830

Merged
merged 1 commit into from
Jan 2, 2024
Merged

keep precision, scale info for databricks decimal #830

merged 1 commit into from
Jan 2, 2024

Conversation

dlawin
Copy link
Contributor

@dlawin dlawin commented Jan 2, 2024 •
edited
Loading

Precision and scale are stripped from DECIMAL(precision, scale) here:

            row_type = "DECIMAL" if info.data_type.startswith("DECIMAL") else info.data_type
            info = attrs.evolve(info, data_type=row_type)

But are parsed below. Retain this info as a quick fix

@dlawin dlawin requested a review from nolar January 2, 2024 21:19
@dlawin dlawin self-assigned this Jan 2, 2024
@dlawin dlawin merged commit 9d2cd65 into master Jan 2, 2024
@dlawin dlawin deleted the databricks_decimal_precision_scale branch January 2, 2024 22:43
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@nolar nolar Awaiting requested review from nolar

Assignees

@dlawin dlawin

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@dlawin