rMW4e4b9997493c: Improve tableExists() for MySQL introduced a more optimized tableExists() method for MySQL, but the function introduced does not honor the configured tablePrefix. This caused various unit tests to start failing, e.g. https://gerrit.wikimedia.org/r/311270, https://gerrit.wikimedia.org/r/311514, https://gerrit.wikimedia.org/r/311515, because the "unittest_" prefix was not being included when things were testing for the unit test tables.
Description
Description
Details
Details
Related Changes in Gerrit:
| Subject | Author | Repo | Branch | Lines +/- | |
|---|---|---|---|---|---|
| Add table prefix in new MySQL tableExists() | Anomie | mediawiki/core | master | +1 -0 |
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | Anomie | T146228 GettingStarted master is now failing CI | |||
| Resolved | Anomie | T146289 DatabaseMysqlBase::tableExists() ignores tablePrefix |
Event Timeline
Comment Actions
Change 312018 had a related patch set uploaded (by Anomie):
Add table prefix in new MySQL tableExists()
Comment Actions
Hmm. The errors being logged (at least for CentralAuth) changed with the fix of this bug, but didn't entirely go away.