Page MenuHomePhabricator
Differential D10800

Apply storage adjustments as part of storage upgrade
ClosedPublic
Actions

Authored by epriestley on Nov 6 2014, 9:56 PM.
Tags
None
Referenced Files
F19004056: D10800.id.diff
Nov 21 2025, 10:20 AM
F18975404: D10800.id25959.diff
Nov 16 2025, 6:35 AM
F18934829: D10800.id25959.diff
Nov 10 2025, 5:01 PM
F18822237: D10800.id25959.diff
Oct 23 2025, 5:14 AM
F18788471: D10800.id25919.diff
Oct 15 2025, 5:02 AM
F18788470: D10800.id25920.diff
Oct 15 2025, 5:02 AM
F18787216: D10800.diff
Oct 14 2025, 10:37 PM
F18783798: D10800.id.diff
Oct 13 2025, 9:36 AM
View All Files
Subscribers
epriestley

Details

Reviewers
btrahan
Maniphest Tasks
T1191: Comments are truncated at first non-base-plane character
Commits
Restricted Diffusion Commit
rPa17a36869254: Apply storage adjustments as part of storage upgrade
Summary

Fixes T1191. I'll write up the changelog with notes about this and open a feedback task for followups.

When you run storage upgrade, automatically run storage adjust afterward. Provide a flag to disable this.

This brings everyone into the utf8mb4 world.

Test Plan

Ran bin/storage upgrade with various flags. Ran bin/storage adjust.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 25919.Nov 6 2014, 9:56 PM
epriestley retitled this revision from to Apply storage adjustments as part of storage upgrade.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
epriestley added a task: T1191: Comments are truncated at first non-base-plane character.
Herald added a subscriber: epriestley. · View Herald TranscriptNov 6 2014, 9:56 PM
Harbormaster completed remote builds in B2971: Diff 25919.Nov 6 2014, 9:57 PM
epriestley updated this revision to Diff 25920.Nov 6 2014, 9:57 PM
  • Make --dryrun win when run as adjust --force --dryrun.
Harbormaster completed remote builds in B2972: Diff 25920.Nov 6 2014, 9:58 PM
epriestley mentioned this in T1191: Comments are truncated at first non-base-plane character.Nov 6 2014, 10:18 PM
btrahan mentioned this in T6350: Diffusion spamming the feed/commits going weird.Nov 6 2014, 10:25 PM
epriestley mentioned this in T6487: Historic Phriction documents do not have mailKeys populated.Nov 7 2014, 12:00 PM
btrahan accepted this revision.Nov 7 2014, 4:54 PM
btrahan edited edge metadata.
This revision is now accepted and ready to land.Nov 7 2014, 4:54 PM
Closed by commit rPa17a36869254: Apply storage adjustments as part of storage upgrade (authored by epriestley, committed by epriestley). · Explain WhyNov 7 2014, 10:25 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 25959

View Options

src/infrastructure/storage/management/workflow/PhabricatorStorageManagementAdjustWorkflow.php

Loading...
View Options

src/infrastructure/storage/management/workflow/PhabricatorStorageManagementUpgradeWorkflow.php

Loading...
View Options

src/infrastructure/storage/management/workflow/PhabricatorStorageManagementWorkflow.php

Loading...
Phabricator · Built by Phacility