Story #11710

[Workbench][Performance] container / container_request show page taking too long to load

Added by Radhika Chippada 6 months ago. Updated 6 months ago.

Status:ResolvedStart date:05/18/2017
Priority:NormalDue date:
Assignee:Radhika Chippada% Done:

100%

Category:-
Target version:2017-05-24 sprint
Story points-Remaining (hours)0.00 hour
Velocity based estimate-

Description

Loading of the pages /containers/qr1hi-dz642-uypqs2exiy7so7l and /container_requests/qr1hi-xvhdp-6490dha4xorrfwz takes more than 60s each.

A couple observations:

  • Each container_uuid of each child is being fetched in a separate request
  • cputime computation is taking a long time as all descendants are being fetched one at a time

Subtasks

Task #11727: Review branch 11710-container-request-show-perfResolvedRadhika Chippada


Related issues

Related to Arvados - Story #11580: [Workbench] Improve format of /container_requests index page Resolved 05/16/2017

Associated revisions

Revision 49a9c33e
Added by Radhika Chippada 6 months ago

closes #11710
Merge branch '11710-container-request-show-perf'

History

#1 Updated by Radhika Chippada 6 months ago

  • Description updated (diff)

#2 Updated by Radhika Chippada 6 months ago

  • Subject changed from [Workbench][Performance] container / container_request show page takes too long to load due to fetching of each associated container in a separate request to [Workbench][Performance] container / container_request show page taking too long to load
  • Description updated (diff)

#3 Updated by Radhika Chippada 6 months ago

  • Status changed from New to In Progress
  • Assignee set to Radhika Chippada
  • Target version set to 2017-05-24 sprint

#4 Updated by Radhika Chippada 6 months ago

  • Description updated (diff)

#5 Updated by Radhika Chippada 6 months ago

  • Description updated (diff)

#6 Updated by Lucas Di Pentima 6 months ago

Just a couple of small comments:

  • File apps/workbench/app/models/container_work_unit.rb
    • Could lines 24 & 26 be unified by using an ‘||’ operator?
      **Line 32: Typo detected on cr.request_container_uuid (also make workbench_units tests fail)

Apart from those, lgtm.

#7 Updated by Radhika Chippada 6 months ago

@ d7e956d1ed2ffeb5b585896e6568481d474bb095

Could lines 24 & 26 be unified by using an ‘||’ operator?

Yes, I cleaned up this and some more

Line 32: Typo detected on cr.request_container_uuid (also make workbench_units tests fail)

Gee thanks for catching this. I corrected this and started tests @ https://ci.curoverse.com/job/developer-run-tests/292/

#8 Updated by Radhika Chippada 6 months ago

  • Status changed from In Progress to Resolved
  • % Done changed from 0 to 100

Applied in changeset arvados|commit:49a9c33ea6119c42ad3d2d585ed7b2f9e4339132.

Also available in: Atom PDF