Skip to content

Force to recreate all diffs on import

Kamil Trzciński requested to merge recreate-all-diffs-on-import into master

What does this MR do?

Today, if the PullRequestImporter fails for whatever reason, for example when pulling git remote, the subsequent run will likely fail, because of ActiveRecord::RecordNotUnique. This MR makes us drop old diffs and recreate them, as method doing that suggests: #insert_or_replace.

What are the relevant issue numbers?

Resolves https://gitlab.com/gitlab-org/gitlab-ce/issues/59353

Does this MR meet the acceptance criteria?

Edited by Yorick Peterse

Merge request reports