iFixit / charge

A tool to upload static images for the site
3 stars 2 forks source link

Improve performance #19

Closed djmetzle closed 4 years ago

djmetzle commented 4 years ago

Charge has been slow lately. Very slow. "Too slow to use" slow.

After some printf debugging, i found out that we were making multiple S3 head-object calls during each route handler due to creating new "Reference" objects for every metadata access. Some light caching reduces the number of S3 API calls to the minimum necessary.

Also, i retouch the control scripts to make them a touch more ergonomic.

cr_req 1 qa_req 0

CC @addison-grant @davidrans

davidrans commented 4 years ago

CR :coffee:

andyg0808 commented 4 years ago

CR :+1: on 09f06bb