Page MenuHomePhabricator

Fix quickstart.sql for old MySQL
ClosedPublic

Authored by epriestley on Mar 2 2015, 4:13 PM.
Tags
None
Referenced Files
F19286218: D11929.id28745.diff
Tue, Dec 23, 6:21 AM
F18896802: D11929.id28735.diff
Nov 7 2025, 4:28 PM
F18835748: D11929.diff
Oct 26 2025, 6:31 PM
F18628121: D11929.diff
Sep 16 2025, 5:38 AM
F18505015: D11929.id.diff
Sep 5 2025, 12:20 AM
F18496627: D11929.diff
Sep 4 2025, 5:34 PM
F18110627: D11929.id28735.diff
Aug 11 2025, 9:49 PM
F18084636: D11929.diff
Aug 5 2025, 4:32 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