From 05d453ec38b10a022ea6db77867957e7115b9b35 Mon Sep 17 00:00:00 2001 From: Tom Clegg Date: Tue, 21 Mar 2017 12:49:06 -0400 Subject: [PATCH] 7709: Do not set job_readable when it won't be used. --- .../api/app/controllers/arvados/v1/nodes_controller.rb | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/services/api/app/controllers/arvados/v1/nodes_controller.rb b/services/api/app/controllers/arvados/v1/nodes_controller.rb index 5e2404e62c..023d2ff888 100644 --- a/services/api/app/controllers/arvados/v1/nodes_controller.rb +++ b/services/api/app/controllers/arvados/v1/nodes_controller.rb @@ -46,10 +46,12 @@ class Arvados::V1::NodesController < ApplicationController @objects = model_class.where('last_ping_at >= ?', db_current_time - 1.hours) end super - job_uuids = @objects.map { |n| n[:job_uuid] }.compact - assoc_jobs = readable_job_uuids(job_uuids) - @objects.each do |node| - node.job_readable = assoc_jobs.include?(node[:job_uuid]) + if @select.nil? or @select.include? 'job_uuid' + job_uuids = @objects.map { |n| n[:job_uuid] }.compact + assoc_jobs = readable_job_uuids(job_uuids) + @objects = @objects.each do |node| + node.job_readable = assoc_jobs.include?(node[:job_uuid]) + end end end -- 2.30.2