- Issue created by @rajeshreeputra
- 🇮🇳India shivam_tiwari
@rajeshreeputra Can you please give more details here related to this issue. I checked from my side, but not getting any issue on update hook.
- 🇭🇺Hungary attilatilman
There is one missing step to reproduce the error. If I understand correctly, then the site already applied a patch from this issue 🐛 General error: 1205 Lock wait timeout exceeded; try restarting transaction RTBC before it got merged into the Autosave form module main branch.
The problem here is, that if someone used a patch to apply the changes before the MR got merged and released, then the index which is added, already on the DB table. Since the index is already there in the DB, update hook should check, is the index already there, if not, add it, otherwise skip adding the index.
This is a specific situation, but I think update hook should take care of this, to make sure every site can apply the changes properly.I'll create a quick fix shortly.
- Merge request !233497311: Check if index exist before adding it to table. → (Merged) created by attilatilman
- First commit to issue fork.
- 🇩🇪Germany hchonov 🇪🇺🇩🇪🇧🇬
@attilatilman, yes this makes sense, thank you. Merged.
Automatically closed - issue fixed for 2 weeks with no activity.