Skip to content

Fix Query Planner able to find struct field with capital letters #16664

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

dttung2905
Copy link
Contributor

Which issue does this PR close?

What changes are included in this PR?

My way of fixing is to follow exact match first which is the current way, then fall back to case-insensitive match if there is no exact match. I'm relatively new to Rust, coming from Golang background so your feedbacks are super valuable for me

I also tested out the test case in the github issue

❯ ./target/debug/datafusion-cli
DataFusion CLI v48.0.0
> select thing.fooBar from 'test.json';
+-------------------------+
| test.json.thing[foobar] |
+-------------------------+
| not going to find me    |
+-------------------------+
1 row(s) fetched. 
Elapsed 0.009 seconds.

> \q
❯ cat test.json
{"thing": {"fooBar": "not going to find me"}}

@github-actions github-actions bot added common Related to common crate functions Changes to functions implementation labels Jul 2, 2025
Signed-off-by: dttung2905 <[email protected]>
@github-actions github-actions bot added the sql SQL Planner label Jul 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
common Related to common crate functions Changes to functions implementation sql SQL Planner
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Query Planner Unable to Find Struct Fields with Capital Letters
1 participant