View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0017944 | mantisbt | performance | public | 2014-12-04 07:08 | 2015-03-15 19:58 |
Reporter | dregad | Assigned To | vboctor | ||
Priority | high | Severity | major | Reproducibility | always |
Status | closed | Resolution | fixed | ||
Product Version | 1.3.0-beta.1 | ||||
Target Version | 1.3.0-beta.2 | Fixed in Version | 1.3.0-beta.2 | ||
Summary | 0017944: Regression from 1.2.x: slow performance when editing issues | ||||
Description | Since we upgraded the tracker to 1.3.0dev, it has become very painful to edit issues, as the page takes up to 5 seconds to load. This is due to Mantis systematically and in most cases unnecessarily loading the 'Reporter' selection list, which on our tracker is very big (28k+ users). This also negatively impacts server performance. In 1.2.x, the current reported was displayed as a hyperlink, and an [Edit] area was available to display the selection list when needed, which made editing issues much faster. Until a proper solution to the underlying performance issue with trackers having very large number of users can be found, the 1.2.x behavior should be reverted. | ||||
Additional Information | Regression from 0011826. See attached screenshots | ||||
Tags | No tags attached. | ||||
Attached Files | |||||
Note: by reverted, I don't mean that we should add back the ajax api... |
|
Pull Request: This change solves the performance issue and provide behavior equivalent to 1.2.x. Once this is done, we can do another change to leverage more sophisticated implementation (e.g. jquery auto-complete). |
|
Many thanks for that fix @vboctor |
|
MantisBT: master b751f3b1 2014-12-13 12:10 Committer: dregad Details Diff |
Don't expand reporters by default in update page For performance reasons with update issue pages, we should revert back to 1.2.x behavior where we don't populate the combobox of reporters by default. This is in order to reduce database load and improve performance for instances with large number of reporters. This change is also designed to make sure that search engine crawlers don't expand such list and hence avoiding the database load. In the future, we should use jquery auto-complete or similar control. We could also possibly add a configuration option or cache a session variable that determines whether to expand the list by default. This can be useful for instances with small number of reporters. Fixes 0017944 |
Affected Issues 0017944 |
|
mod - bug_update_page.php | Diff File | ||
mod - javascript/common.js | Diff File |