Idea #10245
open[Performance][Workbench] Computing user notifications in topnav is expensive. Can the user's properties be used instead?
Added by Radhika Chippada about 8 years ago. Updated 9 months ago.
Updated by Radhika Chippada about 8 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)"}
Updated by Tom Morris about 7 years ago
- Target version set to Arvados Future Sprints
Updated by Ward Vandewege over 3 years ago
- Target version deleted (
Arvados Future Sprints)