Closed Bug 276790 Opened 20 years ago Closed 20 years ago

Use persistent db connections

Categories

(addons.mozilla.org Graveyard :: Administration, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: Bugzilla-alanjstrBugs, Assigned: cso)

Details

Attachments

(1 file)

We should be using mysql_pconnect instead of mysql_connect.  Already fixed on
server.
Changes have already been applied to the server, but we should be doing this in cvs.
Those changes also were reversed on the site. :-)
Attached patch patch v1Splinter Review
Use mysql_pconnect in core/dbconfig.php (If bug 276792 is checked in, this is
the only file that needs to change as this is the only connection line).
Assignee: nobody → colin.ogilvie
Status: NEW → ASSIGNED
Attachment #171578 - Flags: first-review?(Bugzilla-alanjstrBugs)
I need to find out from the sysadmins if this is really what we want to do.
kveton says: After further review, pconnect is not scalable enough
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → WONTFIX
Comment on attachment 171578 [details] [diff] [review]
patch v1

Clearing review request - no point in having it on a WONTFIX bug.
Attachment #171578 - Flags: first-review?(Bugzilla-alanjstrBugs)
Product: addons.mozilla.org → addons.mozilla.org Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: