Change perfherder's signatures endpoint to be more efficient



3 years ago
3 years ago


(Reporter: wlach, Assigned: wlach)




(1 attachment)

1. Remove subtest_signatures from the return value (it's super slow to generate this property, because it requires a database query for every signature with subtests). Instead, add a parent_signature and has_subtests property to each return value (the latter will require an addition to the database schema).
2. Make it optionally possible to only fetch tests without a parent signature (many views only need these signatures, so it's wasteful to fetch everything).
Created attachment 8735558 [details] [review]
[treeherder] wlach:1260110 > mozilla:master

Comment 2

3 years ago
Commits pushed to master at
Bug 1260110 - Add a "has subtests" property to performance series

This will be useful for some views where we don't know off the bat
whether a signature has subtests or not (e.g. the e10s dashboard,
the graphs view) and thus whether to show a "show subtests button"
Bug 1260110 - Re-work the signature fetching API to be faster

* Remove subtest_signatures property (super slow)
* Add a parent_signature property to signatures with a parent signature
* Add option to only fetch signatures without a parental signature
* Update clients to use the new API, for increased speed/awesomeness
:jmaher was away so i just landed this change, after careful testing
Last Resolved: 3 years ago
Resolution: --- → FIXED
Duplicate of this bug: 1234868
You need to log in before you can comment on or make changes to this bug.