-
Notifications
You must be signed in to change notification settings - Fork 16
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
Create collection mixed casing fix #1117
Conversation
…om/stargate/data-api into create-collection-mixed-casing-fix
LGTM |
...in/java/io/stargate/sgv2/jsonapi/service/operation/model/impl/CreateCollectionOperation.java
Show resolved
Hide resolved
if (!indexingDenyAll()) { | ||
String existKeys = | ||
"CREATE CUSTOM INDEX IF NOT EXISTS %s_exists_keys ON \"%s\".\"%s\" (exist_keys) USING 'StorageAttachedIndex'"; | ||
apprender |
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.
Add a comment to explain why we are using quoted identifiers, comments like this make it less likely another person will change it later.
I would also consider a helper function or lambda to quote the identifier rather than have lots of \"
in a string which can easily become unbalanced.
Comment also for the create table statement.
Example "Using CQL quoted identifiers to preserve the case of the identifiers in the CQL schema, and enforce case sensitive comparisons. This is expected behaviour for the Data API."
What this PR does:
Create collection mixed casing fix
Which issue(s) this PR fixes:
Fixes #1110
Checklist