Skip to content

Remove transaction opening for non-basic search count

Madelein van Niekerk requested to merge 465980-search-count-transaction into master

What does this MR do and why?

Removes opening a transaction for search counts introduced in Reduce timeouts on SearchController#count to 5s (!59435 - merged) if the call is not doing a database count. If the search type is advanced or zoekt, the count will be fetched from Elasticsearch or Zoekt which are external calls and can take a while to complete. In those cases we shouldn't have an open database transaction hanging.

Fixes #465980 (closed).

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #465980 (closed)

Merge request reports