Additional orders allows PG to use indexes (#1075)

Author: piotr-travisci

lib/travis/api/v3/queries/repository.rb

@@ -44,15 +44,15 @@ def find!
     def by_slug
       owner_name, repo_name = slug.split('/')
       Models::Repository.where(
-        "(lower(vcs_slug) = ? "\
+        "(lower(repositories.vcs_slug) = ? "\
         "or (lower(repositories.owner_name) = ? and lower(repositories.name) = ?)) "\
         "and lower(repositories.vcs_type) = ? "\
         "and repositories.invalidated_at is null",
         slug.downcase,
         owner_name.downcase,
         repo_name.downcase,
         provider.downcase + 'repository'
-      ).order("updated_at desc").first
+      ).order("updated_at desc, vcs_slug asc, owner_name asc, name asc, vcs_type asc").first
     end
 
     def provider

lib/travis/model/repository.rb

@@ -76,7 +76,7 @@ class Repository < Travis::Model
     without_invalidated.where(
       "(LOWER(repositories.owner_name) = ? AND LOWER(repositories.name) = ?) OR LOWER(vcs_slug) = ?",
       owner_name.downcase, repo_name.downcase, "#{slug.downcase}"
-    ).order('id DESC')
+    ).order('id DESC, owner_name ASC, name ASC, vcs_slug ASC')
   }
   scope :search, ->(query) {
     query = query.gsub('\\', '/')