Page MenuHomePhabricator

Move out transaction management out of Database class
Closed, ResolvedPublic

Description

This includes a complex logic of handling transactions. It really doesn't need to be there.

Event Timeline

Change 755786 had a related patch set uploaded (by Ladsgroup; author: Amir Sarabadani):

[mediawiki/core@master] rdbms: Introduce TransactionManager class to move out the logic

https://gerrit.wikimedia.org/r/755786

Ladsgroup triaged this task as Medium priority.Jan 20 2022, 8:32 PM
Ladsgroup moved this task from Triage to In progress on the DBA board.

Change 755786 merged by jenkins-bot:

[mediawiki/core@master] rdbms: Introduce TransactionManager class to move out the logic

https://gerrit.wikimedia.org/r/755786

Change 757984 had a related patch set uploaded (by Ladsgroup; author: Amir Sarabadani):

[mediawiki/core@master] rdbms: Move more code from Database to TransactionManager

https://gerrit.wikimedia.org/r/757984

Change 757984 merged by jenkins-bot:

[mediawiki/core@master] rdbms: Move more internal code from Database to TransactionManager

https://gerrit.wikimedia.org/r/757984

Change 760664 had a related patch set uploaded (by Ladsgroup; author: Amir Sarabadani):

[mediawiki/core@master] rdbms: Move write transaction reporting to TransactionManager

https://gerrit.wikimedia.org/r/760664

Change 760664 merged by jenkins-bot:

[mediawiki/core@master] rdbms: Move write transaction reporting to TransactionManager

https://gerrit.wikimedia.org/r/760664

Change 762436 had a related patch set uploaded (by Ladsgroup; author: Amir Sarabadani):

[mediawiki/core@master] rdbms: Move more code from Database to TransactionManager

https://gerrit.wikimedia.org/r/762436

Change 762436 merged by jenkins-bot:

[mediawiki/core@master] rdbms: Move more code from Database to TransactionManager

https://gerrit.wikimedia.org/r/762436

Change 763964 had a related patch set uploaded (by Ladsgroup; author: Amir Sarabadani):

[mediawiki/core@master] rdbms: Migrate $this->trxFname to TransactionManager

https://gerrit.wikimedia.org/r/763964

Change 763964 merged by jenkins-bot:

[mediawiki/core@master] rdbms: Migrate $this->trxFname to TransactionManager

https://gerrit.wikimedia.org/r/763964

Change 764466 had a related patch set uploaded (by Ladsgroup; author: Amir Sarabadani):

[mediawiki/core@master] rdbms: Move another batch of transaction management code

https://gerrit.wikimedia.org/r/764466

Change 764466 merged by jenkins-bot:

[mediawiki/core@master] rdbms: Move another batch of transaction management code

https://gerrit.wikimedia.org/r/764466

Change 766188 had a related patch set uploaded (by Ladsgroup; author: Amir Sarabadani):

[mediawiki/core@master] rdbms: Migrate TransactionProfiler calls to TransactionManager

https://gerrit.wikimedia.org/r/766188

Change 766188 merged by jenkins-bot:

[mediawiki/core@master] rdbms: Migrate TransactionProfiler calls to TransactionManager

https://gerrit.wikimedia.org/r/766188

Change 768004 had a related patch set uploaded (by Ladsgroup; author: Amir Sarabadani):

[mediawiki/core@master] rdbms: Move transaction callbacks out of Database to TransactionManager

https://gerrit.wikimedia.org/r/768004

Change 768004 merged by jenkins-bot:

[mediawiki/core@master] rdbms: Move transaction callbacks out of Database to TransactionManager

https://gerrit.wikimedia.org/r/768004

Change 768366 had a related patch set uploaded (by Ladsgroup; author: Amir Sarabadani):

[mediawiki/core@master] rdbms: Simplify TransactionManager class a bit now the code has moved

https://gerrit.wikimedia.org/r/768366

Change 768366 merged by jenkins-bot:

[mediawiki/core@master] rdbms: Simplify TransactionManager class a bit now the code has moved

https://gerrit.wikimedia.org/r/768366

Ladsgroup moved this task from In progress to Done on the DBA board.