[ARVADOS] updated: 0e32c4ee6281dd4a3ae186b75dd7c94c8be508c8
Git user
git at public.curoverse.com
Sun Mar 27 16:47:53 EDT 2016
Summary of changes:
.../views/pipeline_instances/_running_component.html.erb | 10 ----------
services/api/app/controllers/arvados/v1/jobs_controller.rb | 4 ++--
services/api/app/models/job.rb | 13 +++++++------
services/api/test/unit/job_test.rb | 1 -
4 files changed, 9 insertions(+), 19 deletions(-)
via 0e32c4ee6281dd4a3ae186b75dd7c94c8be508c8 (commit)
via adad241671bd37f76b5dcefad1e01914f9797214 (commit)
via 6c01579495dd6a26ee9b05aa0accd136094d453e (commit)
from 087ad4d01176703f22fde4cf053427e562890c16 (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 0e32c4ee6281dd4a3ae186b75dd7c94c8be508c8
Merge: 087ad4d adad241
Author: Brett Smith <brett at curoverse.com>
Date: Sun Mar 27 16:47:07 2016 -0400
Merge branch '8800-queue-query'
Closes #8800, #8809.
commit adad241671bd37f76b5dcefad1e01914f9797214
Author: Brett Smith <brett at curoverse.com>
Date: Sun Mar 27 16:43:55 2016 -0400
8800: Document the new queue_position implementation.
For the benefit of future readers.
diff --git a/services/api/app/models/job.rb b/services/api/app/models/job.rb
index b7f1d9f..afaae26 100644
--- a/services/api/app/models/job.rb
+++ b/services/api/app/models/job.rb
@@ -78,6 +78,12 @@ class Job < ArvadosModel
end
def queue_position
+ # We used to report this accurately, but the implementation made queue
+ # API requests O(n**2) for the size of the queue. See #8800.
+ # We've soft-disabled it because it's not clear we even want this
+ # functionality: now that we have Node Manager with support for multiple
+ # node sizes, "queue position" tells you very little about when a job will
+ # run.
state == Queued ? 0 : nil
end
-----------------------------------------------------------------------
hooks/post-receive
--
More information about the arvados-commits
mailing list