Details
Details
- Reviewers
chad - Maniphest Tasks
- T11672: Evaluate persistent connections from HTTP contexts
- Commits
- rPf8c22252686e: Use persistent database connections from web contexts
Added this to a page:
$tables = array( new PhabricatorUser(), new ManiphestTask(), new DifferentialRevision(), new PhabricatorRepository(), new PhabricatorPaste(), ); $ids = array(); foreach ($tables as $table) { $conn = $table->establishConnection('r'); $cid = queryfx_one( $conn, 'SELECT CONNECTION_ID() cid'); $ids[get_class($table)] = $cid['cid']; } var_dump($ids);
Reloaded the page a bunch of times and saw no reissued connections (the pool seems to keep a particular connection bound to a particular database), but did see connection reuse across requests.
That is, across reloads the same connection IDs appeared, but the same connection ID never appeared twice in the same request. This is what we want.
Also googled for issues with persistent connections, but everything I found was unconcerning and obscure (local variables and other very complex state that we don't use), and a bunch of the docs are reassuring (transactions, etc., get reset properly).
Diff Detail
Diff Detail
- Repository
- rP Phabricator
- Lint
Lint Not Applicable - Unit
Tests Not Applicable