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

mysql not support FOR UPDATE OF #94

Closed
liu3734 opened this issue Jul 30, 2020 · 4 comments · Fixed by #95
Closed

mysql not support FOR UPDATE OF #94

liu3734 opened this issue Jul 30, 2020 · 4 comments · Fixed by #95
Labels
bug

Comments

@liu3734
Copy link

liu3734 commented Jul 30, 2020

When i use myqsl backends . raise this Exception
django_apscheduler/models.py at 155
image
image
@jcass77
Copy link
Owner

jcass77 commented Jul 30, 2020

Duplicate of #93.

@jcass77 jcass77 closed this as completed Jul 30, 2020
@jcass77
Copy link
Owner

jcass77 commented Jul 31, 2020

I was wrong, this isn't an exact duplicate of #93. As noted in that issue however, MySQL does not appear to support using the of parameter in select_for_update.

I think the parameter is probably superfluous in this case as we are not also selecting any related objects via select_related in the QuerySet, so we should be able to just remove it.

@jcass77 jcass77 reopened this Jul 31, 2020
@jcass77 jcass77 mentioned this issue Jul 31, 2020
Merged
@liu3734
Copy link
Author

liu3734 commented Jul 31, 2020

Thanks @jcass77

1 similar comment
@jveeru
Copy link

jveeru commented Jul 31, 2020

Thanks @jcass77

@jcass77 jcass77 mentioned this issue Aug 6, 2020
Closed
@7ooL 7ooL mentioned this issue Aug 9, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Better support for MariaDB and MySQL
3 participants
@liu3734 @jcass77 @jveeru