Skip to content
This repository was archived by the owner on Apr 26, 2024. It is now read-only.
This repository was archived by the owner on Apr 26, 2024. It is now read-only.

Add better method to repopulate room stats than poking the DB (or remove docs). #8246

@erikjohnston

Description

@erikjohnston

As per #8238 we have docs that say to repopulate the room stats table you can insert populate_stats_process_rooms into the background job table. This is a bit icky as a) its not obvious that background jobs are a "public API" and b) if we add further background jobs that depend on populate_stats_process_rooms those will not get re-run (which might lead to bugs).

This feels like something that should be an admin API, or some other form of explicit API, to avoid the above issues.

(Note: we do have tests that manually inserting populate_stats_process_rooms does the Right Thing, but since its not obvious that its a "public API" the tests may just be modified, which is what allowed the regression in #8238)

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions