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

[Docs] Clarify "Large Projects" section #1765

Merged
merged 1 commit into from
Mar 2, 2019

Conversation

gbrova
Copy link
Contributor

@gbrova gbrova commented Jan 20, 2019

This PR clarifies the documentation about large projects.

If my understanding is correct:

  • Using slim_handler increases disk usage, not memory requirements.
  • We are still limited to a maximum size of 512 MB, because AWS restricts the size of /tmp.

 - `slim_handler` increases disk usage, not memory.
 - We are still limited to a maximum size of 512 MB.
@coveralls
Copy link

Coverage Status

Coverage remained the same at 72.806% when pulling 21a2dc0 on gbrova:gbrova/docs-large-projects into 4cf80b2 on Miserlou:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage remained the same at 72.806% when pulling 21a2dc0 on gbrova:gbrova/docs-large-projects into 4cf80b2 on Miserlou:master.

@cl1ent
Copy link
Contributor

cl1ent commented Feb 12, 2019

according to the docs, the limit for deployment package (zip) uploads from s3 is at 250mb (or more specific 262144000 bytes) and not 512mb. its even smaller for direct uploads.

see here and here

@gbrova
Copy link
Contributor Author

gbrova commented Feb 13, 2019

Yes, that is true.

However, if my understanding is correct, the purpose of the slim_handler flag is to overcome that restriction.

Reading through #538, seems this is implemented by copying the zip to /tmp. So, we are still restricted on space, but the restriction is slightly more lax -- according to your first link and also the discussion in 510, the limit for /tmp seems to be 512MB.

It would be nice if this limit was documented somewhere, because I initially (incorrectly) assumed that I can deploy arbitrarily large functions using slim_handler.

Please let me know if I'm misunderstanding something.

@jneves jneves merged commit 2bcdb50 into Miserlou:master Mar 2, 2019
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

Successfully merging this pull request may close these issues.

4 participants