Page MenuHomePhabricator

Provide "--output" flags for "bin/storage renamespace"
ClosedPublic

Authored by epriestley on Aug 17 2016, 3:12 PM.
Tags
None
Referenced Files
F15490240: D16410.id39461.diff
Fri, Apr 11, 3:35 PM
F15487970: D16410.id39468.diff
Thu, Apr 10, 6:32 PM
F15484140: D16410.id.diff
Wed, Apr 9, 4:33 PM
F15479639: D16410.diff
Tue, Apr 8, 8:33 AM
F15467629: D16410.id39461.diff
Thu, Apr 3, 5:04 PM
F15431812: D16410.id.diff
Mon, Mar 24, 2:38 PM
Unknown Object (File)
Mar 3 2025, 5:52 PM
Unknown Object (File)
Feb 18 2025, 11:36 AM
Subscribers
None

Details

Summary

Ref T6996. Depends on D16407. This does the same stuff as D16407, but for bin/storage renamespace. In particular:

  • Support writing directly to a file (so we can get good errors on failure).
  • Support in-process compression.

Also add support for reading out of a storage dump subprocess, so we don't have to do a dump-to-disk + renamespace + compress dance and can just stream out of MySQL directly to a compressed file on disk.

This is used in the second stage of instance exports (see T7148).

It would be nice to share more code with bin/storage dump, and possibly to just make this a flag for it, although we still do need to do the file-based version when importing (vs exporting). I figured that was better left for another time.

Test Plan

Ran bin/storage renamespace --live --output x --from A --to B --compress --overwrite and similar commands.

Verified that a compressed, renamespaced dump came out of the other end.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Provide "--output" flags for "bin/storage renamespace".
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Aug 17 2016, 3:55 PM
This revision was automatically updated to reflect the committed changes.