Fix a badly-performing migration (!7483) · Merge requests · GitLab.org / GitLab FOSS · GitLab

Do not update/delete: Banner broadcast message test data

Do not update/delete: Notification broadcast message test data

The source project of this merge request has been removed.

Nick Thomas requested to merge (removed):24506-fix-timing-out-migration into master Nov 15, 2016

What does this MR do?

Replace a complex UPDATE ... SELECT with a simpler SELECT ; UPDATE. Mea culpa. Introduced in https://gitlab.com/gitlab-org/gitlab-ce/issues/22699

Are there points in the code the reviewer needs to double check?

Why was this MR needed?

The existing SQL isn't performant.

Screenshots (if relevant)

Does this MR meet the acceptance criteria?

Tests
- All builds are passing
Conform by the merge request performance guides
Conform by the style guides
Branch has no merge conflicts with master (if it does - rebase it please)

What are the relevant issue numbers?

Closes #24506 (closed)

Introduced by #22699 (closed)