I cannot flush all caches on the /admin/settings/performance page anymore -- all memory is exhausted by advagg_css_compress/parse.inc, even if I increase the memory limit to extreme sizes:

[Sun Feb 27 22:29:38 2011] [error] PHP Fatal error:  Allowed memory size of 167772160 bytes exhausted (tried to allocate 228 bytes) in /MYPATH/sites/all/modules/advagg/advagg_css_compress/parse.inc on line 785, referer: https://MYSITE/admin/settings/performance
[Sun Feb 27 22:36:20 2011] [error] PHP Fatal error:  Allowed memory size of 199229440 bytes exhausted (tried to allocate 16 bytes) in /MYPATH/sites/all/modules/advagg/advagg_css_compress/parse.inc on line 977, referer: https://MYSITE/admin/settings/performance
CommentFileSizeAuthor
#9 advagg-1075238-9.patch4.07 KBmikeytown2
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mikeytown2’s picture

Component: Code » CSS Compression

This is an issue with the csstidy lib. Looks like I need to test for compressibility before attempting; like I do with jsmin+. Or it could be leaking memory... I'm going to guess memory leaks.

Vacilando’s picture

Got the latest dev now and ran /update.php. The process broke and finished with this hard error on the screen: Fatal error: Allowed memory size of 167772160 bytes exhausted (tried to allocate 16 bytes) in /MYPATH/sites/all/modules/advagg/advagg_css_compress/parse.inc on line 375

mikeytown2’s picture

Disable the css compression for now. On my test file it only shaved 1k off of a 200k file.

Vacilando’s picture

Correction: I got confused by the new package folder naming etc. so that what I put above was an error generated from an earlier dev. The latest dev however fails the same way, only with a different line nr:

Fatal error: Allowed memory size of 167772160 bytes exhausted (tried to allocate 125609 bytes) in /MYPATH/sites/all/modules/advagg/advagg_css_compress/output.inc on line 198

mikeytown2’s picture

Title: Fatal error - allowed memory exhausted » Fatal error - allowed memory exhausted: css tidy library
mikeytown2’s picture

Need to see if the updated lib leaks memory
https://github.com/Cerdic/CSSTidy

Vacilando’s picture

OK, I had to switch off the "Compress CSS" module because of the fatal memory leaks when running system_cron, update.php, performance page cache flush, advagg cache flush, etc. With the sub-module switched off everything works. I am willing to switch it on and test if there's a likely solution, of course -- please update this thread.

mikeytown2’s picture

#1082598: Use BatchAPI for the Rebuild and flush options on the config page will workaround this issue. It's not the ideal fix but this should prevent PHP from bombing out.

mikeytown2’s picture

Status: Active » Fixed
FileSize
4.07 KB

This issue should now be worked around with this patch that has been committed.

Vacilando’s picture

It seems these errors have indeed gone. I'll reopen in case they manifest again.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.