At the sprint the issue about our current 10ms main loop timeout came

up, and it was suggested the problem most likely have been fixed in
HEAD. And even if it hasn't been fixed it's something which should be
fixed rather than plastered over by spinning around on a short timeout
when there is no activity..

This backs out the change to revert the change..

Alex commit reversing the change said:

revno: 8332
committer: rousskov
branch nick: HEAD
timestamp: Tue 2007-07-24 01:55:21 +0000
message:
  Reversed bug #2011 fix because it may slow down ICAP, BodyPipe, and other code
  using zero-delay events to implement "asynchronous" calls.

  The code should probably be rewritten (a) to avoid any waiting/blocking when
  there are ready events and (b) to allow waiting longer when there are no ready
  events.

diff --git a/src/EventLoop.cc b/src/EventLoop.cc
index cc2af543109daf8c4d8625e9cca3968a53b0ad49..dc45f68d8d12099e8b6f48547a1619d6a990e84f 100644 (file)
--- a/src/EventLoop.cc
+++ b/src/EventLoop.cc
@@ -103,7 +103,7 @@ EventLoop::runOnce()
     bool sawActivity = false;
     runOnceResult = true;
     error = false;
-    loop_delay = 10; /* 10 ms default delay */
+    loop_delay = 1000; /* 1s default delay */
 
     AsyncEngine *waitingEngine = primaryEngine;
     if (!waitingEngine && !engines.empty())