[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