-
Notifications
You must be signed in to change notification settings - Fork 46
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
Support UPDATE LIMIT #93
Conversation
Thank you @kozer! I left a few minor comments but this looks good otherwise. It would also be lovely to add the same support for the DELETE queries if you wouldn't mind? The |
Can we proceed with this one, as we consider it a blocker for Studio release, and open a new ticket for the DELETE statement? |
Co-authored-by: Mukesh Panchal <[email protected]>
@kozer nice, getting closer! I went ahead and pushed two failing test cases – let's make sure they work |
…LECT but no WHERE
I had an idea how to fix those two test cases so I went ahead and did that |
Fixes #27
Proposal
Wraps WHERE clauses in UPDATE queries in
rowid in ( SELECT rowid from ...WHERE...)
to support MySQL'sLIMIT
andORDER BY
clauses that are unsupported in SQLite.The implementation in
trunk
was throwing an error when this case occurred. ( For example when a user tries to add Woo plugin ).This pr aims to fix that.