Pim-community-dev: config_prod.yml hardcodes apcu cache pour la doctrine

Créé le 6 avr. 2016  ·  5Commentaires  ·  Source: akeneo/pim-community-dev

Nous avons une belle sélection de pilotes de cache pour la doctrine, pourquoi en coderions-nous en dur ?

Peut-être qu'il devrait être déplacé pour devenir l'un des paramètres de parameters.yml pour l'application. La demande d'extraction est assez évidente, mais discutons-en avant d'en soumettre une.

Edit : clarifier mon anglais.

Commentaire le plus utile

@jmleroux merci pour une mise à jour rapide. Désolé, je n'ai peut-être pas été aussi clair dans ma déclaration, je ne veux pas que Memcached soit l'option par défaut. J'aimerais avoir cela configurable, pas codé en dur.

Tous les 5 commentaires

Bonjour @Xobb ,

Memcached ne devrait pas être une dépendance dure.
Cette personnalisation est assez simple et devrait peut-être être documentée, mais pas définie par défaut.
Pour être honnête, si les métadonnées de doctrine étaient mises en cache dans un tableau à la place d'APC, je ne suis pas sûr que cela aurait un grand impact sur les performances.

Je suis tout à fait d'accord avec @Xobb , cette configuration est essentiellement une variable d'environnement et doit être déplacée vers le fichier parameters.yml.

@jmleroux merci pour une mise à jour rapide. Désolé, je n'ai peut-être pas été aussi clair dans ma déclaration, je ne veux pas que Memcached soit l'option par défaut. J'aimerais avoir cela configurable, pas codé en dur.

Bonjour @xobb ,

Après peu de discussion en interne, nous pensons que cette variable est plus liée à la configuration du projet que ce dont Akeneo PIM a besoin. Vous pouvez l'ajouter à votre parameters.yml dans votre projet mais je ne pense pas que nous devrions l'ajouter à tous les projets. De plus, l'ajout de paramètres à notre fichier parameters.yml les fera apparaître dans notre commande interactive d'installation post-compositeur, ce qui peut être déroutant pour les nouveaux utilisateurs. Enfin, beaucoup de paramètres sont comme celui-ci et l'ajouter à notre parameters.yml pourrait en quelque sorte ouvrir la porte à beaucoup de paramètres.

Comme ces fichiers sont versionnés dans votre projet, vous pouvez facilement le faire pour répondre à vos besoins.

J'espère que ma réponse est claire et pas trop "Pas question Jose" :)

Meilleures salutations,

Julien

@juliensnz

Merci pour votre réponse. Nous avons définitivement trouvé notre chemin à travers cela. Certaines choses ne valent pas la peine d'être produites et celle-ci en fait partie, c'est pourquoi je n'ai jamais fait de pull request. Merci pour la perspicacité et un produit génial que vous créez.

Salutations,
Paul Chubatyy

Cette page vous a été utile?
0 / 5 - 0 notes

Questions connexes

pardahlman picture pardahlman  ·  4Commentaires

lpxavi picture lpxavi  ·  3Commentaires

ronthedrummer picture ronthedrummer  ·  3Commentaires

ronn0 picture ronn0  ·  4Commentaires

ewallteam picture ewallteam  ·  6Commentaires