Closed Bug 408766 Opened 17 years ago Closed 17 years ago

Apache::DBI is a premature optimization that causes trouble

Categories

(Bugzilla :: Database, defect)

3.0.2
defect
Not set
normal

Tracking

()

RESOLVED FIXED
Bugzilla 3.0

People

(Reporter: mkanat, Assigned: mkanat)

References

Details

Attachments

(1 file)

Currently Bugzilla requires and uses Apache::DBI to maintain connections to the database.

However, connecting to the database only takes a tiny fraction of a second, and there isn't load enough on any Bugzilla database anywhere to justify it. 

On the other hand, in certain circumstances the database handles seem to stay open after a request ends, leading to weird "server went away" errors from MySQL.

So overall, we don't need it and it seems to be causing us problems.
Attached patch v1Splinter Review
Assignee: database → mkanat
Status: NEW → ASSIGNED
Attachment #293617 - Flags: review?
Attachment #293617 - Flags: review? → review?(justdave)
Attachment #293617 - Flags: review?(justdave) → review+
Flags: approval?
Flags: approval3.0?
<justdave> and you'll be happy to know that we got the error messages working on the cluster, and there have been no ocurrences whatsoever of "Lost connection to MySQL server during query" since our config change yesterday :)

(He means the change made in this patch.)
Flags: approval?
Flags: approval3.0?
Flags: approval3.0+
Flags: approval+
tip:

Checking in mod_perl.pl;
/cvsroot/mozilla/webtools/bugzilla/mod_perl.pl,v  <--  mod_perl.pl
new revision: 1.8; previous revision: 1.7
done
Checking in Bugzilla/DB.pm;
/cvsroot/mozilla/webtools/bugzilla/Bugzilla/DB.pm,v  <--  DB.pm
new revision: 1.107; previous revision: 1.106
done
Checking in Bugzilla/Install/Requirements.pm;
/cvsroot/mozilla/webtools/bugzilla/Bugzilla/Install/Requirements.pm,v  <--  Requirements.pm
new revision: 1.41; previous revision: 1.40
done
Checking in docs/xml/installation.xml;
/cvsroot/mozilla/webtools/bugzilla/docs/xml/installation.xml,v  <--  installation.xml
new revision: 1.148; previous revision: 1.147
done

3.0 branch:

Checking in mod_perl.pl;
/cvsroot/mozilla/webtools/bugzilla/mod_perl.pl,v  <--  mod_perl.pl
new revision: 1.5.2.2; previous revision: 1.5.2.1
done
Checking in Bugzilla/DB.pm;
/cvsroot/mozilla/webtools/bugzilla/Bugzilla/DB.pm,v  <--  DB.pm
new revision: 1.93.2.4; previous revision: 1.93.2.3
done
Checking in Bugzilla/Install/Requirements.pm;
/cvsroot/mozilla/webtools/bugzilla/Bugzilla/Install/Requirements.pm,v  <--  Requirements.pm
new revision: 1.29.2.2; previous revision: 1.29.2.1
done
Checking in docs/xml/installation.xml;
/cvsroot/mozilla/webtools/bugzilla/docs/xml/installation.xml,v  <--  installation.xml
new revision: 1.136.2.7; previous revision: 1.136.2.6
done
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Keywords: relnote
Resolution: --- → FIXED
Whiteboard: [relnote 3.0.3]
Blocks: 441592
relnoted for 3.0.3 long ago, removing relnote keyword.
Keywords: relnote
Whiteboard: [relnote 3.0.3]
You need to log in before you can comment on or make changes to this bug.