Story #10245

[Performance][Workbench] Computing user notifications in topnav is expensive. Can the user's properties be used instead?

Added by Radhika Chippada almost 3 years ago. Updated almost 2 years ago.

Status:
New
Priority:
Normal
Assigned To:
-
Category:
-
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:
Story points:
-

History

#1 Updated by Radhika Chippada almost 3 years ago

Every page load as well as dashboard refresh at 15s intervals results in computing user notifications ( @@notification_tests ).

This is especially expensive in su92l, but not too cheap in qr1hi either (see log below).

It would be desirable that we remember the fact that the user has created a pipeline_instance and a collection in user properties (similar to getting_started notification) and use this information instead rather than computing this for every page load / refresh.

We might be able to accomplish this by setting these properties the first time they are found when computing @@notification_tests (rather than overloading server side when these objects are created).

su92l

production.log:{"method":"POST","path":"/arvados/v1/collections","format":"*/*","controller":"arvados/v1/collections","action":"index","status":200,"duration":78188.8,"view":0.72,"db":78132.13,"params":{"api_token":"m51fv1765fxl8qny8bwd4t3g5fw6kftqaqgck82g5p09k39x5","reader_tokens":"[false]","current_request_id":"1475530528-710836597","_method":"GET","where":"{\"created_by\":\"su92l-tpzed-2hxfl1bjoe64dpr\"}","limit":"1","offset":"0","_profile":"true"},"@timestamp":"2016-10-03T21:38:13Z","@version":"1","message":"[200] POST /arvados/v1/collections (arvados/v1/collections#index)"}
production.log:{"method":"POST","path":"/arvados/v1/pipeline_instances","format":"*/*","controller":"arvados/v1/pipeline_instances","action":"index","status":200,"duration":2158.64,"view":1.14,"db":1963.41,"params":{"api_token":"m51fv1765fxl8qny8bwd4t3g5fw6kftqaqgck82g5p09k39x5","reader_tokens":"[false]","current_request_id":"1475530528-710836597","_method":"GET","where":"{\"created_by\":\"su92l-tpzed-2hxfl1bjoe64dpr\"}","limit":"1","offset":"0","_profile":"true"},"@timestamp":"2016-10-03T21:38:16Z","@version":"1","message":"[200] POST /arvados/v1/pipeline_instances (arvados/v1/pipeline_instances#index)"}
qr1hi

{"method":"POST","path":"/arvados/v1/collections","format":"*/*","controller":"arvados/v1/collections","action":"index","status":200,"duration":303.61,"view":0.46,"db":290.66,"params":{"api_token":"2asumwa75rz2gu5331pshrqy5aebysooa658zxsh4w387bk7rc","reader_tokens":"[false]","current_request_id":"1475870355-516042097","_method":"GET","where":"{\"created_by\":\"qr1hi-tpzed-ktpvhqu89qoib9f\"}","limit":"1","offset":"0","_profile":"true"},"@timestamp":"2016-10-07T20:01:27Z","@version":"1","message":"[200] POST /arvados/v1/collections (arvados/v1/collections#index)"}
{"method":"POST","path":"/arvados/v1/pipeline_instances","format":"*/*","controller":"arvados/v1/pipeline_instances","action":"index","status":200,"duration":28.73,"view":0.49,"db":13.61,"params":{"api_token":"2asumwa75rz2gu5331pshrqy5aebysooa658zxsh4w387bk7rc","reader_tokens":"[false]","current_request_id":"1475870355-516042097","_method":"GET","where":"{\"created_by\":\"qr1hi-tpzed-ktpvhqu89qoib9f\"}","limit":"1","offset":"0","_profile":"true"},"@timestamp":"2016-10-07T20:01:28Z","@version":"1","message":"[200] POST /arvados/v1/pipeline_instances (arvados/v1/pipeline_instances#index)"}

#2 Updated by Tom Morris almost 2 years ago

  • Target version set to Arvados Future Sprints

Also available in: Atom PDF