HomePhabricator

Allow Owners packages to have arbitrarily long names

Description

Allow Owners packages to have arbitrarily long names

Summary:

  • Change column type from sort128 to sort.
  • Remove originalName. This column is unused. Long ago, we used it to generate a Thread-Topic header for mail, but just use PHIDs now (the value just needs to be stable for a given object, users normally don't see it).

Test Plan:

  • Created a package with a beautifully long name. Magnificent!
  • Grepped for originalName / getOriginalName(), found no Owners hits.
  • Verified that there isn't any name-length validation code to remove.

Screen Shot 2017-04-27 at 3.07.27 PM.png (1×1 px, 173 KB)

Reviewers: chad, amckinley

Reviewed By: chad

Differential Revision: https://secure.phabricator.com/D17798

Details

Provenance
epriestleyAuthored on Apr 27 2017, 10:07 PM
epriestleyPushed on Apr 28 2017, 1:04 AM
Reviewer
chad
Differential Revision
D17798: Allow Owners packages to have arbitrarily long names
Parents
rP85ff1d5c2df0: Reduce the impact of `bin/storage dump`
Branches
Unknown
Tags
Unknown
Build Status
Buildable 16729
Build 22318: Run Core Tests