Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[runtime] Support if in delete statement #13

Closed
pingginp opened this issue Mar 8, 2019 · 2 comments
Closed

[runtime] Support if in delete statement #13

pingginp opened this issue Mar 8, 2019 · 2 comments
Milestone
0.13.0

Comments

@pingginp
Copy link
Owner

pingginp commented Mar 8, 2019 •
edited
Loading

https://docs.datastax.com/en/cql/3.3/cql/cql_reference/cqlDelete.html

DELETE FROM cycling.cyclist_name 
WHERE id =e7ae5cf3-d358-4d99-b900-85902fda9bb0 
if firstname='Alex' and lastname='Smith';
pingginp added a commit that referenced this issue Mar 10, 2019
@pingginp
[runtime] Support if in delete statement #13
2684311
@pingginp
Copy link
Owner Author

there is another issue Eq is only supported on partition columns, for non partition columns, it is not supported, so the generator needs to be changed as well

A quick hack would be allow eq on columns, which is actually a valid case to support IF statements ...

pingginp added a commit that referenced this issue Mar 10, 2019
@pingginp
[gen] Allow Eq on all columns for DELTE IF #13
bb26635 
- previously only primary key and those with index support it
@pingginp pingginp added this to the 0.13.0 milestone Mar 10, 2019
pingginp added a commit that referenced this issue Mar 10, 2019
@pingginp
Merge pull request #14 from pingginp/runtime/delete-if
292abab 
[runtime] Support if in delete statement #13
@pingginp
Copy link
Owner Author

fixed in #14

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
0.13.0
Development

No branches or pull requests
1 participant
@pingginp