GitHub user mars opened a pull request:
https://github.com/apache/incubator-predictionio/pull/394
Extend request timeout for REST API
We've found the default 20-second REST API request timeout is too short for
our batch-prediction use cases. We're running PredictionIO on Heroku which has
its own [timeout starting at
30-seonds](https://devcenter.heroku.com/articles/limits#http-timeouts). So we'd
prefer a more generous or easily configurable timeout to allow Heroku's routing
layer to impose & track this limit in the platform layer.
I investigated how to configure this and found [Spray
`application.conf`](http://spray.io/documentation/1.2.4/spray-can/configuration/).
This PR simply increases the timeout.
I would love guidance on how we might extract this config into an
environment variable or a value in `pio-env.sh`.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/mars/incubator-predictionio
extend-request-timeout
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-predictionio/pull/394.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #394
----
commit 4b99967b47350e2f3ef25e505bd1f523680d7f64
Author: Mars Hall <[email protected]>
Date: 2017-06-16T18:59:10Z
Extend request timeout for REST API
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
