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

Hot function inactivity timeout #554

Closed
WTFKr0 opened this issue Mar 1, 2017 · 3 comments
Closed

Hot function inactivity timeout #554

WTFKr0 opened this issue Mar 1, 2017 · 3 comments

Comments

@WTFKr0
Copy link

WTFKr0 commented Mar 1, 2017

Hi
What do you think about making the 30 seconds inactivity timeout a parameter for hot functions (http mode) ?
I can think about a function with high overhead in loading or database connection
If this function is called every 2 min from 8am to 9am for example, I think it would be great to tune the timeout efficiently
WDYT ?

@pedronasser
Copy link
Contributor

@WTFKr0, when you say parameter do you mean a route configuration?

I agree in making the hot functions timeout configurable.

@WTFKr0
Copy link
Author

WTFKr0 commented Mar 1, 2017

Yep route configuration

@denismakogon
Copy link
Contributor

@WTFKr0 I assume there's only one way to define timeout:

  • extend HTTP Post to /{app}/routes to include new parameter (let's say, inactivity_timeout), but make it optional (similar to other optional parameter we maintain right now)

Would that make sense @pedronasser @seiflotfy @treeder ?

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