Skip to content

Include all queries in the performance bar summary

Sean McGivern requested to merge include-all-postgres-queries-in-performance-bar-summary into master

What does this MR do?

Rails 5.1's exec_no_cache doesn't call async_exec any more, but exec_params: https://github.com/rails/rails/pull/33188

This means that the DB summary in the performance bar was wrong. The individual query details were still correct (we subscribe to ActiveRecord events for those).

We can remove this once the upstream PR to peek-pg is in a release and we update to use that release: https://github.com/peek/peek-pg/pull/10

Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/61372.

Merge request reports