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

Rename storage layer objects to be more sensible #8020

Closed

clokep opened this issue Aug 3, 2020 · 1 comment · Fixed by #8033

Assignees

Member

clokep commented Aug 3, 2020

This is a recurring conversation so wanted to document some of it.

Some places this was talked about:

The current proposal is:

Rename synapse.storage.data_stores to synapse.storage.databases
Rename Database to DatabasePool

We would still have DataStore classes that make up a database (e.g. database.state or database.main).

The text was updated successfully, but these errors were encountered:

Member Author

clokep commented Aug 3, 2020

It sounds like no one has made complaints about this and it is the current plan.

erikjohnston mentioned this issue

Rename database classes to make some sense #8033

Merged

erikjohnston self-assigned this

erikjohnston closed this as completed in #8033

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.