Use instance cache for v2 API related links

RESOLVED WONTFIX

Status

Mozilla Developer Network
BrowserCompat
RESOLVED WONTFIX
2 years ago
6 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/versions
3. Refresh /api/v2/browsers/1/versions

What happened?
==============
There are 3 SQL requests:

1. SELECT ... FROM "auth_user" ...
2. SELECT COUNT(*) FROM "webplatformccompat_version" ...
3. SELECT ... FROM "webplatformcompat_version" ... LIMIT ...



What should have happened?
==========================
There should be at most 1 SQL query for the auth_user

Is there anything else we should know?
======================================
When loading related items, the "primary" item should be loaded from the instance cache, and then a CachedQuerySet should be created for the follow-on queryset.
(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
Blocks: 1240757
No longer blocks: 996570
(Reporter)

Updated

a year ago
Mentor: jwhitlock@mozilla.com
(Reporter)

Comment 1

6 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: 6 months ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.