Mit keys statt values arbeiten um den speicherbedarf zu reduzieren about cache_warmup HOT 8 CLOSED

Finde ich auch. Instanzen kommen morgen rein und dann gibt’s ein neues Release.

from cache_warmup.

Gerade mal einen Test mit array_unique auf der Website gemacht, auf der der Speicherüberlauf passiert ist.

Items: 8481
Items nach array_unique: 1341
Speicher verbraucht: 528440 bytes

Ist jetzt nicht so wild, oder? Sollten wir den Code wie von @staabm beschrieben anpassen oder lieber so lassen?

from cache_warmup.

Oh, spannend: Wenn man hingegen den kompletten Flow trackt, also vom Beginn aller Queries über das Befüllen des Arrays und der späteren Filterung, ist der Speicherverbrauch etwa so:

Vorher: ~5 MB
Nachher: ~1,8 MB

Das spricht deutlich für Markus’ Anpassung, was? Vor allem, weil sie blitzschnell gemacht ist.

from cache_warmup.

Sorry, doch nicht. Hatte versehentlich die Prüfungen auf $media->isImage() ausgeklammert, und das war leider der wesentliche Faktor, der den Speicher so sehr reduziert hat. Kommt die Prüfung wieder rein, ergibt sich leider kein merklicher Unterschied zwischen der Vorher-Variante und der Nachher-Variante. Beide benötigen ~5 MB.

from cache_warmup.

Entschuldigt bitte, dass ich mir hier einen Wolf schreibe, aber: Verwirft man die Mediainstanz nach jeder Prüfung mittels rex_media::clearInstance($item);, schrumpft der Speicherverbrauch in beiden Varianten auf besagte ~1,8 MB. 🎉

(Siehe Kommentar von @gharlan in 541f1da#commitcomment-20528983)

from cache_warmup.

Dann weg mit den instanzen :-)

from cache_warmup.

Sobald wir erstmal an einem punkte sind wo es für große installation gut funktioniert brauch man ja nicht dann jede kleinigkeit ändern.

Wg instance pool wuerd ich noch reinmachen und sonst nur wirklich gut messbares

from cache_warmup.

WONTFIX, hat aber viele Erkenntnisse gebracht. Danke! :)

from cache_warmup.