Page MenuHomePhabricator

Convert SavedQuery / NamedQuery to text, not bytes
ClosedPublic

Authored by epriestley on Oct 1 2014, 4:57 PM.
Tags
None
Referenced Files
F14060442: D10614.diff
Mon, Nov 18, 1:15 AM
F14048388: D10614.id25488.diff
Thu, Nov 14, 7:40 AM
F14046063: D10614.diff
Wed, Nov 13, 5:34 PM
F14029801: D10614.id25488.diff
Fri, Nov 8, 9:26 PM
F14014745: D10614.id25488.diff
Sun, Nov 3, 6:59 AM
F14012326: D10614.diff
Fri, Nov 1, 10:52 AM
F14005949: D10614.id25488.diff
Sun, Oct 27, 10:22 PM
F13997763: D10614.diff
Thu, Oct 24, 6:04 AM
Subscribers

Details

Summary

Ref T1191. Similar issue to D10613. This column usually has a hash exactly 12 bytes long, but sometimes stores an internal builtin query name like "open", "all", etc. It might be nice to promote those to 12-byte hashes of a consistent length eventually, but for now just make this a variable-length column.

Test Plan

Ran migration, no longer saw issues with reordering builtin saved searches.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Convert SavedQuery / NamedQuery to text, not bytes.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
This revision was automatically updated to reflect the committed changes.