-
Notifications
You must be signed in to change notification settings - Fork 86
Improve next/image performance #155
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
Thank you for the update @lindsaylevine! 🎉 Is there a timeline you can share until better caching / performances becomes available? |
hey @MaximeHeckel! my understanding as of right now is that we may not see the internal caching functionality we're looking for until 1-2 months out :( this is because the team is heads down working on other priorities. if i can help speed this up at all, i will! and if there are any updates, i'll be sure to report them here. in the meantime, we suggest in the README (per netlify/next-on-netlify#138) using a provider like Cloudinary. hope that helps!!! |
Thank you @lindsaylevine! So excited to see this happening 🎉 And no worries, this is a dependency on some personal work I can easily postpone, I'm not in a rush 😄 |
Hey @lindsaylevine! Are there any updates you could give regarding the progress the Netlify team has made on this issue? |
@MaximeHeckel heyo. we're still technically at the same crossroads we were at before. however! we have had many, many conversations internally about the right way to approach caching problems like this one, with a clear proposed solution. can't share too much but just know it is moving, and we're making sure we're doing it right and staying aligned with netlify's philosophy. will be worth the wait! let me know if i can answer anything else <3 |
That's great news thank you @lindsaylevine! |
hey, @MaximeHeckel! we have an experimental feature in development that we think helps here [edit: dang, y'all email fast. removing the details now 💜 ] |
a very big shoutout to jason (@jlengstorf) who's been hugely responsible for all of the work that's gone into next/image support from day one and who's been overall instrumental in helping with next-on-netlify's toughest problems!!! ❤️ |
Hey @jlengstorf that would be amazing 🔥 ! sending you an email right now! |
Super excited about this, thanks to everyone at Netlify 🎉 |
Yesss! Can not wait 😁 I’ve tried this a week back and stumbled across this issue. Stoked that there might be a solution already this quickly! 🚀 I’ve sent you an email Jason. Love to try it out. |
Hey @lindsaylevine, |
Looking forward to the release. Just deployed a Next.js app to Netlify with 2 very small Also noticed that the very same repo, same viewport, on a deployment on Vercel loads instantly and images are optimised in size to half of what Netlify does. I'm puzzled. |
@GBrachetta @juliefleet and everyone - thanks so much for your patience! last week we had two lucky folks from this thread test out our solution. on/around april 13th, we'll be able to finally offer it to everyone. 👀 hang tight! |
I have been watching this thread for a bit and I am super excited! Thanks for all the hard work @lindsaylevine ! |
Thank you @lindsaylevine, looking forward to test it and compare it with Vercel's native solution. Will this require any action from us once it's been released (rebuild/redeploy or reinstallation of the plugin)? |
if you have your plugin installed at a fixed version (i.e. you npm installed it), then you'll need to upgrade the plugin version with the new release (when it happens). if you just have the plugin installed via your .toml file or the UI or it's auto installed on your site, then you're good to go and no action necessary! |
hello all! for those who haven't seen: today netlify announced our first step towards solving the problems ISR/ISG originally set out to solve. https://www.netlify.com/blog/2021/04/14/faster-builds-for-large-sites-on-netlify-with-on-demand-builders-now-in-early-access/ what i can offer you today is a tagged release of our plugin: this release uses on-demand builders to dramatically improve next/image performance by caching next/image requests. it is my hope to ship this as the primary release after a week or two of testing. at that time, i will close this issue :). please let me know if you have any feedback or questions!! <3 |
🎉 🎉 The improvements the team is making on this are just amazing! I was testing one of the other experimental-release for next/image support but the Questions:
However the underlying image is relatively "small" : 370kB, and I can load some slightly heavier images. Any tips to handle these use cases? Could it be because the resolution of this image is fairly high? |
|
|
Agreed! Quite an improvement! Thanks, team! |
This issue follows up netlify/next-on-netlify#70. Initial support for next/image was merged in netlify/next-on-netlify#138. However, that initial support is not performant since the Netlify function will re-run on every request without any caching.
Our next step is to improve this performance with caching. This will depend on internal work at Netlify. Please follow this issue for updates on next/image performance!! :)
The text was updated successfully, but these errors were encountered: