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.

Replace register_noop_background_update with DELETE deltas #12940

@clokep

Description

@clokep

From a thread: #12899 (comment)

It seems that register_noop_background_update(...) could be replaced with DELETE FROM background_updates WHERE name = '....';

This would be nicer since we don't end up with dead code, register_noop_background_update was added in 6b02fc8 and from the conversation in #synapse-dev it seems like the DELETE approach was not considered.

Metadata

Metadata

Assignees

Labels

A-Background-UpdatesFilling in database columns, making the database eventually up-to-dateT-TaskRefactoring, removal, replacement, enabling or disabling functionality, other engineering tasks.

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions