Currently the IDatabase interface deals with all possible database interaction.
The interface currently has ~130 methods, with new ones appearing every week or so.
Many bits of code that use our database classes really don't care about 90% of these methods.
Could we put some thought into potentially splitting the interface?
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| rdbms: IDatabase interface cleanups | mediawiki/core | master | +59 -70 |
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | Ladsgroup | T299691 Break down monster class: Database | |||
| Duplicate | Ladsgroup | T190396 Consider splitting the IDatabase interface |
It's kind of hard to do this in practice, give the use of load balancers and so on. Some stuff can be removed, deprecated, or moved to IMaintainableDatabase though.
Change 421327 had a related patch set uploaded (by Aaron Schulz; owner: Aaron Schulz):
[mediawiki/core@master] rdbms: IDatabase interface cleanups
Change 421327 merged by jenkins-bot:
[mediawiki/core@master] rdbms: IDatabase interface cleanups
I see two good opportunities in IDatabase for splitting:
This is the separation layer we've implemented with IMaintainableDatabase, which we use for queries performed by the installer, db-updater, and some maintenance scripts. In other words, the showing, creating, altering and dropping of databases, tables, indexes, and related responsibilities (such as reading patch files).
This is a proposal for a possible next refactor, which would reduce IDatabase to only the state management of the connection and the logic needed to send and interpret queries.
The stateless logic for building sql text (e.g. "query builder") could be separated. The main reason I'm proposing this is so that we could group together all the stateless utility methods for rdbms-backend specific SQL text manipulation. As well as in context of deprecating raw SQL chunks in queries (see T210206).
Change 745215 had a related patch set uploaded (by Ladsgroup; author: Amir Sarabadani):
[mediawiki/core@master] rdmbs: Start of SQL and SQLInterface to split out of Database
The task was fairly vague as to what the outcome criteria would be. I'll close this and merge it into T299691: Break down monster class: Database.
A summary of the work so far:
Two major classes have been split out from IDatabase by @Ladsgroup over the past few months:
This in addition to removing (not moving) various methods such as: