Page MenuHomePhabricator

CentralNotice: Support MariaDB Strict Mode
Closed, ResolvedPublic4 Estimated Story Points


Currently Strict Mode is enabled on CI. It will be gradually enabled in more places, and will eventually land in production.

CentralNotice phpunit tests fail in this mode, so we're pretty sure CentralNotice won't work with this. We should review all DB queries in CN for potential strict mode violations, and fix as needed.

Related Objects

Event Timeline

AndyRussG renamed this task from CentralNotice: Update to use MariaDB Strict Mode to CentralNotice: Support MariaDB Strict Mode.Sep 13 2016, 10:42 PM
DStrine set the point value for this task to 2.
DStrine changed the point value for this task from 2 to 4.

Change 310457 had a related patch set uploaded (by Ejegg):
WIP MariaDB strict mode

Please do not view this as a criticism, but as an encouragement (thanks for working on this!!!)- not enabling strict mode in the past has created both performance and logical errors on central* database tables.

So I believe that, even if this requires time and effort, its investment will be returned many times.

Change 310457 merged by jenkins-bot:
MariaDB strict mode