Page MenuHomePhabricator

Convert `pushlog` and `refcursor` to BLOB storage
ClosedPublic

Authored by epriestley on Aug 11 2014, 3:51 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Aug 28, 11:33 PM
Unknown Object (File)
Mon, Aug 19, 3:12 AM
Unknown Object (File)
Sun, Aug 18, 8:14 PM
Unknown Object (File)
Sun, Aug 18, 4:53 AM
Unknown Object (File)
Sun, Aug 18, 4:52 AM
Unknown Object (File)
Sun, Aug 18, 4:52 AM
Unknown Object (File)
Sun, Aug 18, 4:20 AM
Unknown Object (File)
Wed, Aug 14, 10:17 PM
Subscribers

Details

Summary

Fixes T5840. Some time ago I incorrectly believed that latin1_bin collation was synonymous with "binary". It is not, and does not permit UTF8 characters outside of BMP, among other sequences.

These two tables currently have LONGTEXT columns which should be LONGBLOB. The table design is explicilty intended to accommodate invalid/unreasonably long ref names, but the collation prevents this from working properly.

After T1191, we'll have a general system for resolving this, but a user hit an issue yesterday (T5840) with a brnach name containing Chinese characters.

Test Plan
  • Tried emoji inserts into both tables, was rebuffed.
  • Ran migration.
  • Performed emoji inserts into both tables.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley retitled this revision from to Convert `pushlog` and `refcursor` to BLOB storage.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
btrahan edited edge metadata.

I just can't believe you didn't test emoji branch names at the beginning. :P

This revision is now accepted and ready to land.Aug 11 2014, 6:34 PM
epriestley updated this revision to Diff 24598.

Closed by commit rP31e1653a4e45 (authored by @epriestley).