Move ListCommitsByOid to OPT_OUT

Acceptence testing was successfull in [#844](https://gitlab.com/gitlab-org/gitaly/issues/844) Part of gitlab-org/gitaly#670

Move ListCommitsByOid to OPT_OUT
2ba01185 · Zeger-Jan van de Weg · eaed588b · 2ba01185 · 2ba01185
Unverified Commit 2ba01185 authored Mar 28, 2018 by Zeger-Jan van de Weg
Hide whitespace changes
Inline Side-by-side

Showing with 7 additions and 1 deletion

zj-opt-out-list-commits-by-oid.yml changelogs/unreleased/zj-opt-out-list-commits-by-oid.yml +5 -0

commit.rb lib/gitlab/git/commit.rb +2 -1

No files found.
--- a/changelogs/unreleased/zj-opt-out-list-commits-by-oid.yml
+++ b/changelogs/unreleased/zj-opt-out-list-commits-by-oid.yml
+---
+title: ListCommitsByOid is executed by Gitaly by default
+merge_request:
+author:
+type: performance
--- a/lib/gitlab/git/commit.rb
+++ b/lib/gitlab/git/commit.rb
@@ -231,7 +231,8 @@ module Gitlab
        # relation to each other. The last 10 commits for a branch for example,
        # should go through .where
        def batch_by_oid(repo, oids)
-          repo.gitaly_migrate(:list_commits_by_oid) do |is_enabled|
+          repo.gitaly_migrate(:list_commits_by_oid,
+                              status: Gitlab::GitalyClient::MigrationStatus::OPT_OUT) do |is_enabled|
            if is_enabled
              repo.gitaly_commit_client.list_commits_by_oid(oids)
            else