The "Low Utilization Instances" block takes a long time to build because it makes a CloudWatch API call for every instance. Introduce caching to reduce the number of API calls.

CommentFileSizeAuthor
#3 screenshot 2020-08-25 20.53.50.png127.79 KByas
#3 screenshot 2020-08-25 20.52.46.png154.75 KByas
#2 3167311-2.patch13.74 KBbaldwinlouie
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

baldwinlouie created an issue. See original summary.

baldwinlouie’s picture

baldwinlouie CreditAttribution: baldwinlouie commented
Status: Active » Needs review
FileSize
3167311-2.patch13.74 KB

Added a configuration option to specify caching when configuring the block. The default is 1 hour of caching. The user has the option of turning the caching of cloud watch data off.

yas’s picture

yas
🇯🇵 he/him
Primary language Japanese
Location California 🇺🇸
CreditAttribution: yas at DOCOMO Innovations, Inc. commented
Status: Needs review » Reviewed & tested by the community
FileSize
screenshot 2020-08-25 20.52.46.png154.75 KB
screenshot 2020-08-25 20.53.50.png127.79 KB

@baldwinlouie

Thank you for the performance tuning. It dramatically reduces the rendering speed down to 1/100 powered by the cache mechanism. I'll merge the patch to 8.x-1.x, 8.x-2.x and 3.x and close this issue as Fixed.

46.png
50.png

  • yas committed afefb2c on 8.x-1.x authored by baldwinlouie 
    Issue #3167311 by baldwinlouie, yas: Introduce caching to the "Low...

  • yas committed 24accee on 8.x-2.x authored by baldwinlouie 
    Issue #3167311 by baldwinlouie, yas: Introduce caching to the "Low...

  • yas committed c7ed23c on 3.x authored by baldwinlouie 
    Issue #3167311 by baldwinlouie, yas: Introduce caching to the "Low...
yas’s picture

yas
🇯🇵 he/him
Primary language Japanese
Location California 🇺🇸
CreditAttribution: yas at DOCOMO Innovations, Inc. commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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