-
Notifications
You must be signed in to change notification settings - Fork 206
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
fix: sql identifier default to case-sensitive #641
Conversation
Codecov Report
@@ Coverage Diff @@
## main #641 +/- ##
==========================================
+ Coverage 66.44% 66.50% +0.06%
==========================================
Files 283 283
Lines 44286 44235 -51
==========================================
- Hits 29424 29418 -6
+ Misses 14862 14817 -45
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
b0ec9e1
to
c185797
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
* bump datafusion * fix breaking changes * fix cow errors * fix error convert * fix clippy * fix clippy again * auto fix * create SqlToRel with false ident * fix table reference * fix rust client * fix fmt
Which issue does this PR close?
Closes #442
Rationale for this change
What changes are included in this PR?
Main code changes are:
SqlToRel
withenable_ident_normalization: false
optionTableReference
fromResolvedTableReference
Other changes are clippy fix.
Are there any user-facing changes?
No
How does this change test
integration tests