Use instance cache for v2 relationship links

RESOLVED WONTFIX

Status

Mozilla Developer Network
BrowserCompat
RESOLVED WONTFIX
2 years ago
9 months ago

People

(Reporter: jwhitlock, Unassigned)

Tracking

Details

(Whiteboard: [bc:infra][bc:milestone=spaceship])

(Reporter)

Description

2 years ago
What did you do?
================
1. Turn on DEBUG and the Django Debug Toolbar, enable cache backend
2. Go to /api/v2/browsers/1/relationships/versions
3. Refresh /api/v2/browsers/1/relationships/versions


What happened?
==============
8 SQL queries and no Cache calls are used to render the response

What should have happened?
==========================
With a warmed cache, 1 SQL query is expected at most, to load the auth user. 1 Cache call should be used to render the response

Is there anything else we should know?
======================================
When a relationships link is followed, a CachedModel should be used for the primary instance.
(Reporter)

Updated

2 years ago
Blocks: 996570
Mentor: jwhitlock@mozilla.com
Whiteboard: [specification][type:bug] → [bc:infra][bc:milestone=spaceship]
(Reporter)

Updated

2 years ago
Mentor: jwhitlock@mozilla.com
(Reporter)

Comment 1

9 months ago
The BrowserCompat project is canceled.  See https://github.com/mdn/browsercompat for current effort. Bulk status change includes the random word TEMPOTHRONE.
Status: NEW → RESOLVED
Last Resolved: 9 months ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.