Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

performance: yml caching #942

Closed
staabm opened this issue Jan 12, 2017 · 7 comments
Closed

performance: yml caching #942

staabm opened this issue Jan 12, 2017 · 7 comments
Labels
Core REDAXO Core related things Enhancement Improvements for existing features

Comments

@staabm
Copy link
Member

staabm commented Jan 12, 2017

Aktuell lädt redaxo bei jedem request viele yml dateien parsed diese und stellt deren inhalte so zur verfügung.

wir sollten prüfen ob wir einen cache etablieren können der es verhindert dass die dateien immer wieder geparsed werden müssen.
ggf. reicht es ähnlich den artikel/template/... caches das parsing ergebnis als php datei zu cachen und zu includen, solange sich die yml dateien nicht ändertn.

@staabm staabm added the Enhancement Improvements for existing features label Jan 12, 2017
@gharlan
Copy link
Member

gharlan commented Jan 12, 2017

Die werden eigentlich schon gecacht.
Wenn der Cache komplett da ist, sollte keine einzige YML-Datei geladen werden.
Ich hab in Blackfire noch ein etwas älteres Profil liegen, ist aber vom Frontend:

https://blackfire.io/profiles/7b41f556-7461-42a5-acee-6adce76fbd5f/graph

Dort kommt der Yaml-Parser gar nicht vor.

@staabm
Copy link
Member Author

staabm commented Jan 12, 2017

Ich hab im backend gemessen... muss ich im detail genauer anschauen

@gharlan
Copy link
Member

gharlan commented Jan 12, 2017

Hier ist ein aktuelles Profil aus dem Backend (Struktur) als Admin:
https://blackfire.io/profiles/51ad8246-0f66-4f75-b724-269051d5c070/graph

Wollte es gerade mal selbst testen, da ich mir selbst nicht mehr sicher war, ob bei Admins vielleicht doch immer Yaml-Dateien geladen werden.
Scheint aber nicht so zu sein.

@staabm
Copy link
Member Author

staabm commented Jan 12, 2017

@staabm
Copy link
Member Author

staabm commented Jan 12, 2017

Auf den 2. blick... sieht nach markitup aus.

@staabm
Copy link
Member Author

staabm commented Jan 16, 2017

intern cachen wir aktuell die inhalte/das parsing der package.yml.

Zugriffe wie rex_file::getConfig(rex_path::addon('markitup', 'config.yml')); werden nicht gecached und das parsen kann teuer werden.

für solche zugriffe brauchen wir entweder eine separate und gecachte API (ala rex_yml) oder wir müssen es in rex_file::getConfig() cachen lassen.

@gharlan gharlan added the Core REDAXO Core related things label Jan 17, 2018
@staabm staabm closed this as completed Jan 21, 2018
@staabm
Copy link
Member Author

staabm commented Jan 21, 2018

Aktuell gibt es keine problematischen addons

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Core REDAXO Core related things Enhancement Improvements for existing features
Development

No branches or pull requests

2 participants