Skip to content

Further remove `if postgresql?` branches

Andreas Brandl requested to merge ab-remove-postgresql-switches into master

What does this MR do?

This change removes code where we branch out by type of database. In https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/29608, we already removed the if mysql?; ... else ... end parts. Here we take the other perspective and remove if postgresql?; ... else ... end branches.

Relates to https://gitlab.com/gitlab-org/gitlab-ce/issues/52442 but there's no other issue for it.

EE MR: https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/14824

Follow-ups

I've kept this change strictly to the above. This led to 3 methods that look like we could refactor and remove them entirely. In order to keep things small, I created those issues and will address this separately:

Does this MR meet the acceptance criteria?

Conformity

Edited by Andreas Brandl

Merge request reports

Loading