-
Notifications
You must be signed in to change notification settings - Fork 356
Escaping case-sensitive columns [DATAJDBC-407] #628
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
Comments
Jens Schauder commented Does this problem still exists with DATAJDBC-381 (i.e. the current 2.0 Milestone)? Make sure you register the proper |
If you would like us to look at this issue, please provide the requested information. If the information is not provided within the next 7 days this issue will be closed. |
Closing due to lack of requested feedback. If you would like us to look at this issue, please provide the requested information and we will re-open the issue. |
…verters. Creating R2dbcCustomConversions without a dialect or StoreConversions can easily lead to misconfiguration that isn't immediately obvious because of missing store simple types. We now deprecated the constructor and have added guidance on how to properly create R2dbcCustomConversions that is associated with a dialect. Closes #628
Pavel Kapyla opened DATAJDBC-407 and commented
If one have an entity with case-sensitive columns (e.g. on Postgres database) DML statements generated by default implementation of CrudRepository product wrong statements.
But binding parameters leads to exception:
because the parameterSource uses column names (with excaping commas) as keys for parameter values.
The same problem was partly addressed in DATAJDBC-334, now columns are correctly escaped, bad parameters problem still exists.
Actually the fix (DATAJDBC-381) solves the issue but padding with "_" is not needed in this case.
I hope it's possible to fix the problem in RC3
Affects: 1.0.10 (Lovelace SR10), 1.1 RC2 (Moore)
Issue Links:
DATAJDBC-381 Using backticks in column names leads to failure during INSERT with MySQL
DATAJDBC-386 Always quote identifiers
The text was updated successfully, but these errors were encountered: