Bugzilla::local_timezone calls DateTime::TimeZone->new(name => 'local') which is very slow. It takes 0.2 second on my machine, for each page call. The local timezone won't change while a process is running, and so there is no need for this timezone object to be rebuilt again and again. It should be stored in process_cache instead of request_cache.
Created attachment 701110 [details] [diff] [review] patch, v1
Assignee: general → LpSolit
Status: NEW → ASSIGNED
Attachment #701110 - Flags: review?(glob)
Comment on attachment 701110 [details] [diff] [review] patch, v1 r=glob
Attachment #701110 - Flags: review?(glob) → review+
Committing to: bzr+ssh://email@example.com/bugzilla/trunk/ modified Bugzilla.pm Committed revision 8556. Committing to: bzr+ssh://firstname.lastname@example.org/bugzilla/4.4/ modified Bugzilla.pm Committed revision 8504.
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.