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

Improve "Fail to merge schema" error messages #3095

Closed
andygrove opened this issue Nov 12, 2022 · 2 comments · Fixed by #3098 or #3113
Closed

Improve "Fail to merge schema" error messages #3095

andygrove opened this issue Nov 12, 2022 · 2 comments · Fixed by #3098 or #3113
Labels
arrow Changes to the arrow crate enhancement Any new improvement worthy of a entry in the changelog good first issue Good for newcomers

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 seeing this error:

ArrowError(SchemaError("Fail to merge schema Field due to conflicting datatype"))

It does not tell me which field has the issue or what the conflicting data types are.

Describe the solution you'd like

I would like the error to give me more information.

Describe alternatives you've considered

Additional context
@andygrove andygrove added good first issue Good for newcomers enhancement Any new improvement worthy of a entry in the changelog labels Nov 12, 2022
@askoa
Copy link
Contributor

askoa commented Nov 13, 2022

I'll pick this up.

@askoa askoa mentioned this issue Nov 13, 2022
Merged
@andygrove andygrove mentioned this issue Nov 15, 2022
Merged
@alamb
Copy link
Contributor

alamb commented Nov 25, 2022

label_issue.py automatically added labels {'arrow'} from #3098

@alamb alamb added the arrow Changes to the arrow crate label Nov 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
arrow Changes to the arrow crate enhancement Any new improvement worthy of a entry in the changelog good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Improve schema mismatch error message
Include field name in merge error message andygrove/arrow-rs
3 participants
@alamb @andygrove @askoa