Page MenuHomePhabricator

Fix quickstart.sql for old MySQL
ClosedPublic

Authored by epriestley on Mar 2 2015, 4:13 PM.
Tags
None
Referenced Files
F14035981: D11929.diff
Sun, Nov 10, 8:09 AM
F14018921: D11929.diff
Tue, Nov 5, 8:34 PM
F13966749: D11929.id28745.diff
Wed, Oct 16, 9:28 AM
F13958981: D11929.id.diff
Mon, Oct 14, 5:15 PM
Unknown Object (File)
Oct 11 2024, 11:44 PM
Unknown Object (File)
Oct 6 2024, 11:29 PM
Unknown Object (File)
Oct 6 2024, 11:29 PM
Unknown Object (File)
Oct 6 2024, 11:29 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