-
Notifications
You must be signed in to change notification settings - Fork 991
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
[20.09] Drop trigger and update time queries #10821
[20.09] Drop trigger and update time queries #10821
Conversation
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.
Thanks for fixing this, I was so worried about this and put so much effort into trying to cause these issues and really just let the project down by failing to produce the deadlocks. I'm really sorry for breaking this and really appreciate you for cleaning it up.
8d44927
to
22a2d7c
Compare
I don't think we currently filter on HDA or HDCA update_time in the UI, so this should work around the observed deadlocks, at the cost of having to re-implement a replacement on 21.01 when we'll merge galaxyproject#8441.
22a2d7c
to
33883fe
Compare
I've updated the PR so that we call |
@mvdbeek Can I merge this forward as it is? |
Yes, but I do think we should probably make that a migration. |
Partially revert commit 33883fe . Also: - Update `update_time` on target (i.e. `history`) table AFTER HDA/HDCA insert/update/delete - Use f-strings instead of templates - Use `CURRENT_TIMESTAMP` also in PostgreSQL (it's equivalent but standard SQL) - Code cleanups
Partially revert commit 33883fe . Also: - Update `update_time` on target (i.e. `history`) table AFTER HDA/HDCA insert/update/delete - Use f-strings instead of templates - Use `CURRENT_TIMESTAMP` also in PostgreSQL (it's equivalent but standard SQL) - Code cleanups
Restore triggers dropped in #10821
I don't think we currently filter on HDA or HDCA update_time in the UI, so this should work around the observed deadlocks, at the cost of having to re-implement a replacement on 21.01 when we'll merge #8441.