Commit f0056cca by Christopher Lee

Merge pull request #148 from edx/clee/increase-timeout

Increased timeout back to 20s. Added comment
parents 339fd755 affd9676
# It is possible that the rack time out here is set to a different value than
# on the edx-platform django_comment_client timeout. An attempt was made to
# move these two values closer together (5s django_client_comment, 6s
# cs_comments_service from 20). This resulted in more reported timeout errors
# on the cs_comments_service side which better reflected the timeout errors the
# django_comment_client. On the downside, the shorter timeout lead to less time
# for processing longer queries in the background. The timeout has been set back
# to 20s. Until these slow queries that benefit from being cached in the
# background are resolved, reducing the timeout is not suggested.
# More conversation at https://github.com/edx/cs_comments_service/pull/146
# -Nov 18th, 2015
require "rack-timeout" require "rack-timeout"
use Rack::Timeout # Call as early as possible so rack-timeout runs before other middleware. use Rack::Timeout # Call as early as possible so rack-timeout runs before other middleware.
Rack::Timeout.timeout = 6 Rack::Timeout.timeout = 20
require './app' require './app'
run Sinatra::Application run Sinatra::Application
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment