UBOdin / mimir

Data-ish exploration through SQL+Uncertainty
http://mimirdb.info
Apache License 2.0
27 stars 13 forks source link

Get rid of Database.init() #341

Closed okennedy closed 4 years ago

okennedy commented 5 years ago

Database's init method exists because there used to be a cyclic dependency: Both backend and database needed to be linked together before either could be initialized. This cyclic dependency no longer exists, and so init serves only one purpose: Initializing (or updating) the metadata schema.

This is something that can/should happen during the Database constructor.

This is a relatively minor rewrite in terms of lines of code, but will need to be carefully tested to avoid breaking anything.