Partial revert #17044 to fix sql performance regression

[csett86]

For details see #17129. Compared to a full revert, we
keep the class method instroduced in #17129, as its used
elsewhere by now

Fixes: #17129
Signed-off-by: Christoph Settgast [email protected]

Pull Request Checklist

• Pull request is based on the develop branch
• Pull request includes a changelog file. The entry should:
  • Be a short description of your change which makes sense to users. "Fixed a bug that prevented receiving messages from other servers." instead of "Moved X method from EventStore to EventWorkerStore.".
  • Use markdown where necessary, mostly for code blocks.
  • End with either a period (.) or an exclamation mark (!).
  • Start with a capital letter.
  • Feel free to credit yourself, by adding a sentence "Contributed by @github_username." or "Contributed by [Your Name]." to the end of the entry.
• Code style is correct
  (run the linters)

[CLAassistant]

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}

[erikjohnston]

@csett86 have you tried this and seen that it helps? The queries should basically be the same so am a bit confused how the revert would help, but if you've confirmed it does then I'll have a deeper look

[csett86]

Yes @erikjohnston, I have deployed this for 10 days now and it helps immensely: #17129 (comment)

This is on a public homeserver with a reasonable amount of rooms and active users, where postgres was maxing out without this revert. See the issue #17129 for details and others reporting the same.

[erikjohnston]

Hi @csett86, I would be very interested to know if the patch in #17169 helps?

[PhieF]

@csett86 I can confirm your mr also fixes my problem, I'm using your patch on a docker build, io is back to normal as well as cpu usage, thanks !

[reivilibre]

We don't currently think this is safe, but we're still looking into how to fix the performance regression in a safe way.

[PhieF]

@reivilibre for now I'm keeping this patch on my server, without it it is fully unusable (both synapse and other services)