Skip to content

Fix reindexing bug when statement_timeout set

Andreas Brandl requested to merge ab/fix-reindexing-statement-timeout-bug into master

What does this MR do?

The disable_statement_timeout helper is a migration helper with a couple branches. If the statement timeout isn't set at all, it does nothing. This has been the case in dev and CI - until this change, where we configure a statement timeout for those envs.

This also adds a non-mocked spec to test reindexing. In combination, this surfaces the problem described in #251084 (closed) in CI.

Plus, we then fix the bug by renaming methods.

Thinking this doesn't warrant a changelog entry - the rake task has just been added without advertising it in documentation (yet).

Does this MR meet the acceptance criteria?

Conformity

Edited by Andreas Brandl

Merge request reports