[5.x]: Asset::getSrcset takes too many database queries
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 45/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- mysql, php
- Área
- backend, databases, performance
Línea de trabajo
Comienza en Asset::getSrcset() y reproduce el problema con los tamaños indicados mientras observas el recuento de consultas a la base de datos del panel de depuración. Rastrea cómo se carga cada tamaño de transformación y, después, verifica que todas las transformaciones se carguen en una sola consulta y que añadir tamaños ya no añada consultas por cada tamaño.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
What happened?
Description
I'm investigating some performance issues with the MySQL server, so I'm trying to reduce the number of database queries required for a page load, especially on a cold cache, because that's where most of the problems occur.
I found that the single largest source for the query count is Asset::getSrcset(). This uses a separate database query for every single size. In fact, I found that every single size takes two database queries. With 5 transforms in two formats (WebP and AVIF), we get 20 database queries for a single image! And that's just for the srcset, there are additional queries for the base transform.
Steps to reproduce
- Where
imageis anAsset, useimage.getSrcset([240, 360, 480, 600, 720, 840, 960])and watch the number of database queries in the debug panel as the number of sizes is increased.
Expected behavior
Asset::getSrcset() should group its queries to avoid the N+1 (or rather, N+2) problem here. All transforms should be loaded in a single query, adding more sizes should not result in additional database queries for each one.
Actual behavior
The number of database queries grows exponentially at an alarming rate with the number of images on a page, and transforms per image.
Craft CMS version
5.8.6
PHP version
8.3
Operating system and version
No response
Database type and version
No response
Image driver and version
No response
Installed plugins and versions
- Lenguaje dominante
- PHP
- Estrellas
- 3.6k
- Forks
- 705
- Merge medio
- 1 d 5 h
- PR fusionados (30 d)
- 134
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de craftcms/cms
-
bug
Dificultad 4/5 3-5 días Aptitud para principiantes 52/100
-
bug
Dificultad 3/5 1-2 días Aptitud para principiantes 68/100
-
bug
-
bug
-
bug
Dificultad 4/5 3-5 días Aptitud para principiantes 30/100
Todos los issues de craftcms/cms
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
getgrav/grav-plugin-api#45 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
RSS-Bridge/rss-bridge#5098 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
phingofficial/phing#2025 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
silverstripe/developer-docs#911 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100