Commit 13c8f7fa authored by Byron Jones's avatar Byron Jones

Bug 702208: TheSchwartz should not open a new db connection for each email

r=mkanat, a=mkanat
parent 2f52dd3d
...@@ -35,6 +35,11 @@ use constant JOB_MAP => { ...@@ -35,6 +35,11 @@ use constant JOB_MAP => {
send_mail => 'Bugzilla::Job::Mailer', send_mail => 'Bugzilla::Job::Mailer',
}; };
# Without a driver cache TheSchwartz opens a new database connection
# for each email it sends. This cached connection doesn't persist
# across requests.
use constant DRIVER_CACHE_TIME => 300; # 5 minutes
sub job_map { sub job_map {
if (!defined(Bugzilla->request_cache->{job_map})) { if (!defined(Bugzilla->request_cache->{job_map})) {
my $job_map = JOB_MAP; my $job_map = JOB_MAP;
...@@ -62,6 +67,7 @@ sub new { ...@@ -62,6 +67,7 @@ sub new {
pass => $lc->{db_pass}, pass => $lc->{db_pass},
prefix => 'ts_', prefix => 'ts_',
}], }],
driver_cache_expiration => DRIVER_CACHE_TIME,
); );
return $self; return $self;
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment