Paths

Table of Contentst

Differential D20743

Straighten out replication/cache behavior in "bin/storage dump"
ClosedPublic
Actions

Authored by epriestley on Aug 28 2019, 3:19 PM.

Tags

None

Referenced Files

	F15265074: D20743.diff
	Tue, Feb 25, 10:19 PM

	F15265018: D20743.diff
	Tue, Feb 25, 10:15 PM

	Unknown Object (File)
	Sun, Feb 9, 8:02 AM

	Unknown Object (File)
	Sun, Feb 9, 8:02 AM

	Unknown Object (File)
	Sun, Feb 9, 8:02 AM

	Unknown Object (File)
	Sun, Feb 9, 8:02 AM

	Unknown Object (File)
	Sun, Feb 2, 1:52 AM

	Unknown Object (File)
	Jan 25 2025, 5:35 AM

Subscribers

None

Details

Reviewers: None
Maniphest Tasks: T13336: "bin/storage dump" option "--for-replica" might not interact properly with "--no-indexes" and PERSISTENCE_CACHE
Commits: rPc6642213d57f: Straighten out replication/cache behavior in "bin/storage dump"

Summary

Fixes T13336.

Prevent --no-indexes from being combined with --for-replica, since combining these options can only lead to heartbreak.
In --for-replica mode, dump caches too. See discussion in T13336. It is probably "safe" to not dump these today, but fragile and not correct.
Mark the "MarkupCache" table as having "Cache" persistence, not "Data" persistence (no need to back it up, since it can be fully regenerated from other datasources).

Test Plan

Ran bin/storage dump with various combinations of flags.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Aug 28 2019, 3:19 PM

Harbormaster completed remote builds in B23317: Diff 49458.Aug 28 2019, 3:21 PM

epriestley requested review of this revision.Aug 28 2019, 3:21 PM

This revision was not accepted when it landed; it landed in state Needs Review.Aug 28 2019, 3:25 PM

Closed by commit rPc6642213d57f: Straighten out replication/cache behavior in "bin/storage dump" (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

epriestley added a commit: rPc6642213d57f: Straighten out replication/cache behavior in "bin/storage dump".

Revision Contents
Changeset List

Path

Size

src/

applications/

cache/

storage/

PhabricatorMarkupCache.php

4 lines

config/

schema/

PhabricatorConfigSchemaSpec.php

10 lines

infrastructure/

storage/

lisk/

4 lines

management/

workflow/

PhabricatorStorageManagementDumpWorkflow.php

23 lines

Diff 49459

src/applications/cache/storage/PhabricatorMarkupCache.php

Loading...

src/applications/config/schema/PhabricatorConfigSchemaSpec.php

Loading...

src/infrastructure/storage/lisk/LiskDAO.php

Loading...

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

Loading...