[ARVADOS] updated: b6552a6b3c1202eafebfc630ee7e9d6c919b811e
Git user
git at public.curoverse.com
Fri Oct 7 15:57:42 EDT 2016
Summary of changes:
.../app/controllers/application_controller.rb | 7 ++--
.../app/views/projects/_show_dashboard.html.erb | 43 +++++++++++++++++----
.../app/views/work_units/_show_component.html.erb | 18 ++++++++-
apps/workbench/config/application.default.yml | 6 +++
apps/workbench/test/helpers/share_object_helper.rb | 8 ++--
.../test/integration/application_layout_test.rb | 5 ++-
.../test/integration/pipeline_instances_test.rb | 8 ++--
apps/workbench/test/integration/users_test.rb | 3 +-
apps/workbench/test/integration/work_units_test.rb | 6 ++-
apps/workbench/test/integration_helper.rb | 18 ++++++---
apps/workbench/test/unit/work_unit_test.rb | 2 +-
build/run-build-packages.sh | 2 +-
sdk/cwl/arvados_cwl/__init__.py | 3 +-
sdk/cwl/setup.py | 2 +-
services/api/app/models/container.rb | 27 +++++++++++++
services/api/app/models/container_request.rb | 12 +++++-
services/api/config/application.default.yml | 7 ++++
.../migrate/20160926194129_add_container_count.rb | 9 +++++
services/api/db/structure.sql | 7 +++-
services/api/test/fixtures/container_requests.yml | 20 ++++++++++
services/api/test/unit/container_request_test.rb | 44 ++++++++++++++++++++--
21 files changed, 217 insertions(+), 40 deletions(-)
create mode 100644 services/api/db/migrate/20160926194129_add_container_count.rb
via b6552a6b3c1202eafebfc630ee7e9d6c919b811e (commit)
via 4d015af8133d6a50ff85dd6a9c3cfbe5c092e43f (commit)
via d68cb1845e82f44ecf126ae4fd1ba3abc00f4463 (commit)
via e9bf0a8dec771a53818ee76026c8bcc93a4722da (commit)
via 23b1c79162bd422f99e4d7a0e491d82da560d91f (commit)
via 304273fabb09ff72e94481a610e84f78776f05e3 (commit)
via f4c5d28ce98099b2f4204b5875449d76609c6a1c (commit)
via 2f86d9ac26cad9cd202ccfe9cf07c31ecbabcf7e (commit)
via 4c8559157a16bf3d2781714fdec0275fb7f53ac6 (commit)
via 38f17f1e09e1eb2af273148744ca193d2541d37e (commit)
via a4ef9968a1b59b99efac934076b8216f3b2f499a (commit)
via 47e42f1129363c2565e69c36ff26ce9c42731fb8 (commit)
via 78995d3bf5b9e160de50318bc83744dda92e4211 (commit)
via a894a61cc9004e184bc63f9f0a785f555d6c6829 (commit)
via 30d4c3f8f1d4d303c66fc568b11c34be7c51f2a8 (commit)
via a605fc29fd7f79b2882625c99a72e998157fa5bc (commit)
via 8317b6e41e86f3e5c988906e0a9965479acf7e51 (commit)
via 929e817fc1c91e1d9a0afcfcab93fe07304d9be5 (commit)
via 068b2c9dbe1e467119680cbab61c62a03ad4fafd (commit)
via 5905f1299f3a55519d1c79991e405ae8d47711f9 (commit)
via eabe4efe7b0964f537f2e09fb0d820272f1c86e7 (commit)
via 986cdafd30a7bef4ccfbd18f808be52b2f6a458e (commit)
via bc67ce8d098c36e999739a9213b29bb72af6f23d (commit)
via 1ce0bd54f40a686959e95e63f7aaa8b233269932 (commit)
via 54cd184005d14ba93e93ae0eeb16ea70ee09a3b6 (commit)
via 49987769d924c1bc77cbdc9e9b182c3e2cc09b2d (commit)
via 3fb81a4db7abbaaa67b7a18d1c4a5ce82bc232dc (commit)
from cc18e9552f5ffb99910b0196e01ceb20d49da24c (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 b6552a6b3c1202eafebfc630ee7e9d6c919b811e
Author: radhika <radhika at curoverse.com>
Date: Fri Oct 7 15:46:30 2016 -0400
10144: Preloading all the collection using one single large query is much slower than making several requests
with a smaller subset of uuids/pdhs when displaying a pipeline instance with many jobs. Hence, make several
requests for 200 collections at a time.
diff --git a/apps/workbench/app/views/work_units/_show_component.html.erb b/apps/workbench/app/views/work_units/_show_component.html.erb
index 2290a65..9bf751f 100644
--- a/apps/workbench/app/views/work_units/_show_component.html.erb
+++ b/apps/workbench/app/views/work_units/_show_component.html.erb
@@ -50,8 +50,22 @@
collections.concat wu.children.collect {|j| j.log_collection}.uniq.compact
collections_pdhs = collections.select {|x| !(m = CollectionsHelper.match(x)).nil?}.uniq.compact
collections_uuids = collections - collections_pdhs
- preload_collections_for_objects collections_uuids if collections_uuids.any?
- preload_for_pdhs collections_pdhs if collections_pdhs.any?
+
+ if collections_uuids.any?
+ start = 0; inc = 200
+ while start < collections_uuids.length
+ preload_collections_for_objects collections_uuids[start, inc]
+ start += inc
+ end
+ end
+
+ if collections_pdhs.any?
+ start = 0; inc = 200
+ while start < collections_pdhs.length
+ preload_for_pdhs collections_pdhs[start, inc]
+ start += inc
+ end
+ end
repos = wu.children.collect {|c| c.repository}.uniq.compact
preload_objects_for_dataclass(Repository, repos, :by_name) if repos.any?
commit 4d015af8133d6a50ff85dd6a9c3cfbe5c092e43f
Merge: cc18e95 d68cb18
Author: radhika <radhika at curoverse.com>
Date: Fri Oct 7 14:37:16 2016 -0400
Merge branch 'master' into 10144-pipeline-display-perf
-----------------------------------------------------------------------
hooks/post-receive
--
More information about the arvados-commits
mailing list