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

Add StateGroupStorage interface#6294

Merged
erikjohnston merged 4 commits into
developfrom
erikj/add_state_storage
Oct 31, 2019
Merged

Add StateGroupStorage interface#6294
erikjohnston merged 4 commits into
developfrom
erikj/add_state_storage

Conversation

@erikjohnston
Copy link
Copy Markdown
Member

@erikjohnston erikjohnston commented Oct 30, 2019

This currently just uses the existing main data store.

Commits are independently reviewable.

(Split out from #6245.)

Copy link
Copy Markdown
Member

@richvdh richvdh left a comment

Choose a reason for hiding this comment

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

lgtm.

Comment thread synapse/storage/state.py Outdated

class EventHandler(BaseHandler):
def __init__(self, hs):
super(EventHandler, self).__init__(hs)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

argless super() is a thing in py3 btw

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Mind. Blown.

Fix docstring

Co-Authored-By: Richard van der Hoff <1389908+richvdh@users.noreply.github.com>
@erikjohnston erikjohnston merged commit dfe0cd7 into develop Oct 31, 2019
@erikjohnston erikjohnston deleted the erikj/add_state_storage branch January 9, 2020 15:47
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