Mage_Catalog_Model_Product_Image :: _ checkMemory

8

En regardant les demandes lentes dans newrelic pour une page de liste de produits, la principale fonction appelée était Mage_Catalog_Model_Product_Image :: _ checkMemory () qui fait un tas de trucs comme la vérification des paramètres ini pour memory_limit ( ini_get('memory_limit'), vérifie la taille de l'image et calcule la mémoire nécessaire pour cela, etc.) sur...

Je viens d'ajouter un return true(sauter les vérifications, Magento a suffisamment de mémoire pour ces processus) dans cette fonction et le graphique ci-dessous montre l'impact.

Quelqu'un d'autre a fait quelque chose à ce sujet? Des idées, des risques?

entrez la description de l'image ici

FlorinelChis
la source
3
J'ai fait la même chose, pas de problème, sauf si les administrateurs du magasin commencent à télécharger des fichiers d'images de monstre.
Vinai
1
J'ai fait exactement ça! Aucun effet indésirable dont j'ai été témoin jusqu'à présent.
philwinkle
Allons-nous soumettre un patch à Magento? :)
FlorinelChis
Un correctif pour Magento 2 pour le rendre configurable serait bien. Le faire par défaut pour "vérifier la limite de mémoire" ne casserait pas la boutique pour les utilisateurs dans des environnements limités comme Alan le mentionne ci-dessous.
Matthias Zeis

Réponses:

1

À part les mises en garde habituelles à propos de la nécessité de fusionner vous-même ces modifications avec les mises à niveau , il ne semble rien dans ces méthodes qui modifie l'état de PHP ou de Magento de manière significative. Ils semblent être strictement là pour aider les utilisateurs qui exécutent Magento dans un environnement à mémoire limitée pour éviter les plantages PHP.

Alan Storm
la source