Drop's the time by quite a bit if using the database cache; seeing a %40 percent speed improvement.

CommentFileSizeAuthor
#8 advagg-2408271-8-fix-js-cache-save.patch523 bytesmikeytown2
#5 advagg-2408271-1-fix-js-settings.patch1.47 KBmikeytown2
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

  • mikeytown2 committed 3b386cc on 7.x-2.x 
    Issue #2408271 by mikeytown2: Do 1 cache_get_multiple instead of 3...
mikeytown2’s picture

mikeytown2 CreditAttribution: mikeytown2 commented
Status: Active » Fixed
mikeytown2’s picture

mikeytown2 CreditAttribution: mikeytown2 commented
Status: Fixed » Needs work

#2408929: Regression for possibly settings in last dev release

  • mikeytown2 committed 5b5d82a on 7.x-2.x 
    Issue #2408271 by mikeytown2: Do 1 cache_get_multiple instead of 3...
mikeytown2’s picture

mikeytown2 CreditAttribution: mikeytown2 commented
Status: Needs work » Needs review
FileSize
advagg-2408271-1-fix-js-settings.patch1.47 KB

Patch has been committed. Will change to fixed once I confirm bug is gone

joelpittet’s picture

joelpittet
Primary language English
Location Vancouver
CreditAttribution: joelpittet commented
Status: Needs review » Reviewed & tested by the community

Yes that did the trick, thank you:)

mikeytown2’s picture

mikeytown2 CreditAttribution: mikeytown2 commented
Status: Reviewed & tested by the community » Fixed

Awesome thanks! Already committed so marking this as fixed

mikeytown2’s picture

mikeytown2 CreditAttribution: mikeytown2 commented
FileSize
advagg-2408271-8-fix-js-cache-save.patch523 bytes

One more thing to fix...

  • mikeytown2 committed 12ac7f4 on 7.x-2.x 
    Issue #2408271 by mikeytown2: Do 1 cache_get_multiple instead of 3...
joelpittet’s picture

joelpittet
Primary language English
Location Vancouver
CreditAttribution: joelpittet commented

Still good;)

Status: Fixed » Closed (fixed)

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