Skip to content

Remove projects join when using ci minutes denormalization

Max Orefice requested to merge mo-add-remove-unused-join-ci-minutes into master

What does this MR do?

Following up !66082 (merged).

This MR removes a join on projects when using ci minutes denormalized which will no longer being used.

Why are we doing this?

We released ci minutes denormalization today and we will release shared runners denormalization next week.

As explained in our EE::BuildQueueService codebase:

If shared runners information is denormalized then the query does not include the join with projects anymore, so we need to add it until we use denormalized ci minutes

This MR takes makes sure we don't join projects anymore when fully using denormalized data.

This MR is behind 2 feature flag :

  • ci_queueing_denormalize_shared_runners_information
  • ci_queueing_denormalize_ci_minutes_information

Does this MR meet the acceptance criteria?

Conformity

Edited by Max Orefice

Merge request reports