[ARVADOS] updated: 0c356edf482e4ef2bb32dddde8eabe562adcfd8a

[Git user]

Summary of changes:
 services/api/app/controllers/application_controller.rb | 3 ---
 1 file changed, 3 deletions(-)

       via  0c356edf482e4ef2bb32dddde8eabe562adcfd8a (commit)
       via  9deb1c897c821937563d863ed07fb5dee9de55b6 (commit)
      from  51fe3482a5253d054e838fb1dcf3982f79650cee (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit 0c356edf482e4ef2bb32dddde8eabe562adcfd8a
Merge: 51fe348 9deb1c8
Author: Tom Clegg <tom at curoverse.com>
Date:   Wed Aug 16 13:06:05 2017 -0400

    Merge branch '11898-no-distinct'
    
    refs #11898
    
    Arvados-DCO-1.1-Signed-off-by: Tom Clegg <tom at curoverse.com>


commit 9deb1c897c821937563d863ed07fb5dee9de55b6
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
--