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

Fix typo where we ended up with multiple WorkerLocksHandler #16220

Merged
merged 2 commits into from
Sep 1, 2023

Conversation

erikjohnston
Copy link
Member

I don't think has caused any actual issues.

Introduced in #15891

Copy link
Contributor

@DMRobertson DMRobertson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess this is safe because we expect there to be multiple such handlers across workers---so having two separate handlers in memory should be okay?

@erikjohnston
Copy link
Member Author

@DMRobertson yeah, pretty much

@erikjohnston erikjohnston marked this pull request as ready for review September 1, 2023 12:22
@erikjohnston erikjohnston requested a review from a team as a code owner September 1, 2023 12:22
@erikjohnston erikjohnston enabled auto-merge (squash) September 1, 2023 12:22
@erikjohnston erikjohnston merged commit 93f2fdd into develop Sep 1, 2023
37 checks passed
@erikjohnston erikjohnston deleted the erikj/fix_missing_cache branch September 1, 2023 12:52
DMRobertson pushed a commit that referenced this pull request Sep 1, 2023
I don't think has caused any actual issues.

Introduced in #15891
bob-beck pushed a commit to openbsd/ports that referenced this pull request Sep 4, 2023
fix a "typo" - matrix-org/synapse#16220

patch from maintainer Renaud Allard
netbsd-srcmastr pushed a commit to NetBSD/pkgsrc that referenced this pull request Sep 8, 2023
# Synapse 1.91.2 (2023-09-06)

### Bugfixes

- Revert [MSC3861](matrix-org/matrix-spec-proposals#3861) introspection cache, admin impersonation and account lock. ([\#16258](matrix-org/synapse#16258))


# Synapse 1.91.1 (2023-09-04)

### Bugfixes

- Fix a performance regression introduced in Synapse 1.91.0 where event persistence would cause an excessive linear growth in CPU usage. ([\#16220](matrix-org/synapse#16220))
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants