[ARVADOS] created: ced3a175a174f7a583758efa7d54e54b968b280c

[Git user]

        at  ced3a175a174f7a583758efa7d54e54b968b280c (commit)


commit ced3a175a174f7a583758efa7d54e54b968b280c
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 b6816d3..703787f 100644
--- a/services/api/app/controllers/application_controller.rb
+++ b/services/api/app/controllers/application_controller.rb
@@ -83,9 +83,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
--