Skip to content

Reduce the per-page size in release index page for reducing the number of markdown post-processing

Shinya Maeda requested to merge optimize-release-page-markdown-rendering into master

What does this MR do?

This MR is to address the performance issue on the Release index page loading by changing the per-page size in release index page.

As the smaller number of entries that need to post-process the description_html, we can expect an immediate performance improvement. For example, in my local environment, The loading time is cut down to half.:

PAGE_SIZE = 20 => 1490 msec

screenshot_222

PAGE_SIZE = 10 => 691 msec

screenshot_221

This per-page-size seems same with GitHub releases that shows 10 releases by default (Example).

Screenshots

Peek_2021-01-08_14-28

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team
Edited by Shinya Maeda

Merge request reports

Loading