Skip to content

Increase statement timeout

Andreas Brandl requested to merge ab/increase-statement-timeout into master

What does this Merge Request do?

This is a follow-up to !1565 (merged). There have been reports of gdk environments running into the statement timeout when running specs, in particular around the DatabaseCleaner.clean_with(:truncation) strategy.

My intent was to set some statement timeout, not to limit this to 15s strictly. I had naiively assumed this would be more than enough, but that's not the case.

So this change increases the timeout to 2 minutes.

We can make this configurable through gdk.yml, but I think it may not be worth it. Happy to add this if needed.

There is gitlab#37311 which talks about the problems with cleaning up the database.

Merge Request checklist

  • Tests added for new functionality. If not, please raise Issue to follow-up.
  • This change is backward compatible. If not, please include steps to communicate to our users.
  • Documentation added/updated, if needed.
  • gdk doctor test added, if needed.
Edited by Andreas Brandl

Merge request reports