Page MenuHomePhabricator

Allow persistent connections to be configured per database host
ClosedPublic

Authored by epriestley on Nov 22 2016, 2:41 PM.

Details

Summary

Ref T11044. Fixes T11672. In T11672, persistent connections seem to work fine, but they can require max_connections and other settings to be raised. Since most users don't need them, make them an advanced option.

Test Plan

Configured persistent connections, loaded some pages, observed persistent connections get used.

Diff Detail

Repository
rP Phabricator
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

epriestley updated this revision to Diff 40716.Nov 22 2016, 2:41 PM
epriestley retitled this revision from to Allow persistent connections to be configured per database host.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
epriestley updated this revision to Diff 40721.Nov 22 2016, 6:03 PM
  • Fix a documentation typo.
chad accepted this revision.Nov 22 2016, 6:11 PM
chad edited edge metadata.
This revision is now accepted and ready to land.Nov 22 2016, 6:11 PM
This revision was automatically updated to reflect the committed changes.