Page MenuHomePhabricator

Update some storage documentation for new adjustment workflows
ClosedPublic

Authored by epriestley on Oct 30 2014, 4:40 PM.

Details

Summary

Ref T1191. General update of this document, which remains mostly accurate. Remove a warning.

Test Plan

Read document.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Update some storage documentation for new adjustment workflows.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
btrahan edited edge metadata.

Another great doc!

src/docs/contributor/database.diviner
19

Does this need to be caveated at all? (e.g. we *do* support MariaDB and Amazon RDS then?)

This revision is now accepted and ready to land.Oct 30 2014, 4:59 PM
src/docs/contributor/database.diviner
19

I'll make this more clear, I was mostly missing a term for "we don't support officially it but we kind-of-sort-of support it", like "tacit support" or "grey support".

src/docs/contributor/database.diviner
19

"Users have reported that Phabricator works with MariaDB and Amazon RDS." Basically states our relatively-weak technological mastery of these things and has a nice flavor to it...?

src/docs/contributor/database.diviner
207

any sort of step a la rP915d9a52f085e0ecb3c00208f1a5809761b76fac you think or is that overkill?

src/docs/contributor/database.diviner
207

I'm going to make bin/storage adjust warn (but not fail) on that, and then make bin/storage upgrade apply bin/storage adjust after upgrading, so the net effect will be that the issue becomes a lot harder to get wrong and a lot less serious someone does.

epriestley edited edge metadata.
  • Clarify MySQL/MariaDB/RDS.
This revision was automatically updated to reflect the committed changes.
polybuildr added inline comments.
src/docs/contributor/database.diviner
13

Small typo? "Phabricator uses the"?