-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
NotImplementedException
- Querying JSON file with duplicate column names crashes CLI
#10751
Closed
1 task done
Closed
1 task done
Labels
Comments
hello-world-bfree
changed the title
Feb 19, 2024
NotImplementedException
- Table created from JSON file with duplicate column names crashes CLI NotImplementedException
- Querying JSON file with duplicate column names crashes CLI
Thanks! Reproduced, we'll take a look. |
I have pushed a partial fix for this in #10881 that correctly renders an exception instead in this case instead of aborting. The JSON issue separately needs to be fixed, however. |
Mytherin
added a commit
that referenced
this issue
Feb 28, 2024
Partially fix #10751: correctly catch exceptions in sqlite3_print_duckbox
Might be related: #11152 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
What happens?
When querying a newline-delimited JSON file with
read_json_auto
DuckDB crashes with aNotImplementedException
:The column name
platform
is just the first of a number of the same kind of duplicates. They're duplicates in the sense that their lower cased representation matches - "Platform" == "platform". This jives with DuckDB elsewhere; i.e.,So rejecting these is consistent. The bug is the lack of error handling or validation.
If the file is altered so that the field names are truly identical -
platform
==platform
instead ofPlatform
andplatform
, we get the expected behavior and no crash:To Reproduce
Create a table with the attached file:
Query the newly created table:
OS:
macOS
DuckDB Version:
0.10.0
DuckDB Client:
CLI
Full Name:
Brandon Freeman
Affiliation:
Hallow
Have you tried this on the latest nightly build?
I have not tested with any build
Have you tried the steps to reproduce? Do they include all relevant data and configuration? Does the issue you report still appear there?
The text was updated successfully, but these errors were encountered: