Page MenuHomePhabricator

Fix quickstart.sql for old MySQL
ClosedPublic

Authored by epriestley on Mar 2 2015, 4:13 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Sep 12, 2:35 PM
Unknown Object (File)
Thu, Sep 12, 2:34 PM
Unknown Object (File)
Thu, Sep 12, 2:34 PM
Unknown Object (File)
Thu, Sep 12, 2:34 PM
Unknown Object (File)
Thu, Aug 29, 4:33 AM
Unknown Object (File)
Tue, Aug 27, 1:22 AM
Unknown Object (File)
Aug 19 2024, 3:23 AM
Unknown Object (File)
Aug 15 2024, 8:18 PM
Subscribers

Details

Summary

Fixes T7422. After the recent fix for "sort" columns, we can end up with invalid SQL in some cases when running quickstart.

In particular, we do "COLLATE binary CHARACTER SET utf8_general_ci" (which is invalid).

Preprocess these so we get "COLLATE utf8 CHARACTER SET utf8_general_ci" (which is valid and correct).

Test Plan

Ran bin/storage upgrade -f --namespace blahblhbaba with and without --disable-utf8mb4.

Diff Detail

Repository
rP Phabricator
Branch
storage2
Lint
Lint Warnings
SeverityLocationCodeMessage
Warningresources/sql/quickstart.sql:290TXT3Line Too Long
Warningresources/sql/quickstart.sql:1267TXT3Line Too Long
Warningresources/sql/quickstart.sql:1276TXT3Line Too Long
Warningresources/sql/quickstart.sql:1290TXT3Line Too Long
Warningresources/sql/quickstart.sql:1364TXT3Line Too Long
Warningresources/sql/quickstart.sql:1392TXT3Line Too Long
Warningresources/sql/quickstart.sql:1686TXT3Line Too Long
Warningresources/sql/quickstart.sql:1727TXT3Line Too Long
Warningresources/sql/quickstart.sql:1744TXT3Line Too Long
Warningresources/sql/quickstart.sql:1820TXT3Line Too Long
Warningresources/sql/quickstart.sql:1915TXT3Line Too Long
Warningresources/sql/quickstart.sql:1987TXT3Line Too Long
Warningresources/sql/quickstart.sql:1988TXT3Line Too Long
Warningresources/sql/quickstart.sql:2472TXT3Line Too Long
Warningresources/sql/quickstart.sql:2503TXT3Line Too Long
Warningresources/sql/quickstart.sql:2538TXT3Line Too Long
Warningresources/sql/quickstart.sql:2547TXT3Line Too Long
Warningresources/sql/quickstart.sql:2603TXT3Line Too Long
Unit
Tests Passed
Build Status
Buildable 4705
Build 4719: [Placeholder Plan] Wait for 30 Seconds