[ARVADOS] created: b459e0fc5b8911198085703188383e964fe6b46f

[Git user]

        at  b459e0fc5b8911198085703188383e964fe6b46f (commit)


commit b459e0fc5b8911198085703188383e964fe6b46f
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
--