Skip to content

Fix N+1 query in MR pipelines. Also fix N+1 testing documentation

Furkan Ayhan requested to merge fa/fix-project-mergerequests-pipelines-nplus1 into master

What does this MR do and why?

This MR fixes the N+1 query problem in the MR pipelines endpoint. (calling "has_warnings?" in serializers. Adding "preload: true" will fix it.)

It also fixes the N+1 testing documentation. (Maybe, we should break these into two separate MRs but I wanted to leave it in this MR to show the problem better). Please see the inline comments in the MR diff.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Furkan Ayhan

Merge request reports