Browse files

Raising send timeout to 5s on HTTP(s) drains

When network or drains show bad behaviour temporarily, low timeouts we
currently have (1 second) ends up killing connections and raising the
retry count of frames. When massive losses are seen, it makes it
difficult to put the blame on logplex's speed at sending logs, or the
drains consumption (or network).

By raising the timeout a bit, we should reduce the reconnection rate and
at the same time make it harder to blame logplex (as an individual node)
for the problems.

This should not have a super significant impact on the drop rate,
however, but possibly a noticeable one.
  • Loading branch information...
1 parent 58487de commit a35010b305e1441532c9e2d9e86ae0202ff8916c @ferd ferd committed Aug 23, 2013
Showing with 1 addition and 1 deletion.
  1. +1 −1 src/logplex_http_drain.erl
@@ -53,7 +53,7 @@
-define(HTTP_VERSION, 'HTTP/1.1').
-define(RECONNECT_MSG, reconnect).
-define(CONNECT_TIMEOUT, 1000).
--define(REQUEST_TIMEOUT, 1000).
+-define(REQUEST_TIMEOUT, 5000).
%% ------------------------------------------------------------------
%% API Function Exports

0 comments on commit a35010b

Please sign in to comment.