Page MenuHomePhabricator

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

Authored by epriestley on Aug 17 2016, 3:12 PM.
Tags
None
Referenced Files
F18114040: D16410.id39468.diff
Wed, Aug 13, 3:29 AM
F18112773: D16410.id.diff
Tue, Aug 12, 8:27 PM
F18108974: D16410.diff
Mon, Aug 11, 10:55 AM
F17818319: D16410.id.diff
Sat, Jul 26, 12:16 AM
F17796530: D16410.diff
Fri, Jul 25, 1:02 AM
Unknown Object (File)
Jul 5 2025, 10:51 AM
Unknown Object (File)
Jun 18 2025, 7:52 PM
Unknown Object (File)
Jun 18 2025, 1:31 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.