Skip to content
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

Drawbacks? #5

Open
jescalan opened this issue Mar 5, 2022 · 2 comments
Open

Drawbacks? #5

jescalan opened this issue Mar 5, 2022 · 2 comments

Comments

@jescalan
Copy link

jescalan commented Mar 5, 2022

Hey friends!

This library is super amazing and I am a huge fan of the work being done here. I was just curious if it might be helpful though to write out some of the drawbacks, if there are any, in the readme. For example, since it's using a lambda layer, I assume this means it will cut into the space available per lambda. The normal limit is ~250mb, and node image layers are ~100mb. Additionally, is there a performance penalty incurred when loading node through the layer on a cold boot? Is there anything else I'm missing?

Not trying to be a downer at all here, I just want to make sure that the tradeoffs are clear for anyone thinking about adopting 😁

@tjanczuk
Copy link
Contributor

tjanczuk commented Mar 9, 2022

Thanks @jescalan! I will keep this issue open until we get around to updating the readme with those considerations. Quickly, however, there appear to be no performance penalty for cold start, but there is a performance penalty for creating the function itself - creating functions with layers takes more time, on the order of seconds.

@bestickley
Copy link

Have there been any official tests run to prove no perf penalty for cold start? I'm not worried so much about perf penalty creating the function, but would like to see numbers for cold starts. Thanks! Great project!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants