Page MenuHomePhabricator

Never sever non-cluster database; write more read-only documentation
ClosedPublic

Authored by epriestley on Apr 11 2016, 1:50 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Apr 16, 11:55 AM
Unknown Object (File)
Thu, Apr 11, 9:29 AM
Unknown Object (File)
Wed, Apr 10, 4:02 PM
Unknown Object (File)
Mon, Apr 8, 9:00 AM
Unknown Object (File)
Wed, Mar 27, 6:44 AM
Unknown Object (File)
Fri, Mar 22, 12:37 PM
Unknown Object (File)
Thu, Mar 21, 8:27 AM
Unknown Object (File)
Feb 10 2024, 10:11 AM
Subscribers
None

Details

Summary

Ref T4571. Write more of the missing documentation sections and clarify a few things.

Since the "replicating master" check needs a special permission, imposes a performance penalty, is probably very difficult to misconfigure, and likely not a big deal anyway, just drop the idea of trying to automatically detect + prevent it. We still show if it's an issue on the status page, provided we have permission to check.

When you don't have any cluster databases configured, never stop trying to connect to the default master database. We might want to do this eventually as load reduction, but just don't muddy the waters too much for now while things stabilize.

Test Plan
  • Tested functionality in cluster, non-cluster, and degraded-cluster modes.
  • Used status console to monitor a health check cycle.
  • Read docs.

Diff Detail

Repository
rP Phabricator
Branch
readonly13
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 11649
Build 14573: Run Core Tests
Build 14572: arc lint + arc unit

Event Timeline

epriestley retitled this revision from to Never sever non-cluster database; write more read-only documentation.
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.Apr 11 2016, 3:42 PM
This revision was automatically updated to reflect the committed changes.