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

ARROW-11209: [Rust] DF - Better error message on unsupported GROUP BY #9163

Closed
wants to merge 1 commit into from

Conversation

ovr
Copy link
Contributor

@ovr ovr commented Jan 11, 2021

I am getting error without details which type is not supported inside GROUP BY:

External error: Arrow error: External error: Internal error: Unsupported GROUP BY data type.

I added the column's DataType to the error message to make the investigation easier.

@github-actions
Copy link

@andygrove
Copy link
Member

andygrove commented Jan 11, 2021

Hi @ovr could you write a more descriptive PR description? The PR description becomes the commit message when the PR is merged so we want it to explain exactly what the change is/was. Thanks!

@ovr ovr changed the title ARROW-11209: [Rust] DF - Provider better error message on unsupported… ARROW-11209: [Rust] DF - Better error message on unsupported GROUP BY Jan 11, 2021
@ovr
Copy link
Contributor Author

ovr commented Jan 11, 2021

Thanks, @andygrove, for the notice. I've updated the description, and I will use a more useful description in my PRs.

@alamb
Copy link
Contributor

alamb commented Jan 11, 2021

BTW @ovr I am not sure what type you were grouping on, but I had started working on support for String dictionary support for grouping but have gotten distracted. Just FYI -- please don't feel like you need to change what you are working on I just wanted to throw that out there.

@ovr
Copy link
Contributor Author

ovr commented Jan 12, 2021

@alamb I think this problem caused with our fork and Float support, I've prepared a PR with support for Float32/Float64 #9175

But anyway, it's better to get an error message with details which type is not supported, instead of trying to understand it from SQL and DB.

@alamb alamb closed this in 28e7388 Jan 12, 2021
GeorgeAp pushed a commit to sirensolutions/arrow that referenced this pull request Jun 7, 2021
I am getting error without details which type is not supported inside GROUP BY:

External error: Arrow error: External error: Internal error: Unsupported GROUP BY data type.

I added the column's DataType to the error message to make the investigation easier.

Closes apache#9163 from ovr/issue-11209

Authored-by: Dmitry Patsura <zaets28rus@gmail.com>
Signed-off-by: Andrew Lamb <andrew@nerdnetworks.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants