[ARVADOS] created: e7e1e4ff9cfcc888e4132ba5820220a735eb385a

Git user git at public.curoverse.com
Thu Jul 20 11:42:46 EDT 2017


        at  e7e1e4ff9cfcc888e4132ba5820220a735eb385a (commit)


commit e7e1e4ff9cfcc888e4132ba5820220a735eb385a
Author: Tom Clegg <tom at curoverse.com>
Date:   Fri Jun 23 15:17:23 2017 -0400

    11898: Remove superfluous uniq(&:id) call
    
    Avoids adding DISTINCT to SQL query, which causes Postgres to do
    expensive things.
    
    Arvados-DCO-1.1-Signed-off-by: Tom Clegg <tom at curoverse.com>

diff --git a/services/api/app/controllers/application_controller.rb b/services/api/app/controllers/application_controller.rb
index 3a66f53..a3d8f08 100644
--- a/services/api/app/controllers/application_controller.rb
+++ b/services/api/app/controllers/application_controller.rb
@@ -87,9 +87,6 @@ class ApplicationController < ActionController::Base
   end
 
   def index
-    if @select.nil? || @select.include?("id")
-      @objects = @objects.uniq(&:id)
-    end
     if params[:eager] and params[:eager] != '0' and params[:eager] != 0 and params[:eager] != ''
       @objects.each(&:eager_load_associations)
     end

-----------------------------------------------------------------------


hooks/post-receive
-- 




More information about the arvados-commits mailing list