| commit | 4dd1d6f2c713aefadee746e72279e618a57c689d | |
| author | Eric Wong <normalperson@yhbt.net> | |
| Fri, 2 Aug 2013 18:35:24 +0000 (2 18:35 +0000) | ||
| committer | Eric Wong <normalperson@yhbt.net> | |
| Sat, 10 Aug 2013 07:04:53 +0000 (10 07:04 +0000) | ||
| tree | 10d3a52212480a5a7862ebb000c184fd688e9e6c | treesnapshot (tar.gz zip) |
| parent | 11e3cdca95ad8e0621549af6a2320de24d3a6f9f | commitdiff |
monitor: remove misleading error message for timeout
The timeout we're removing includes time spent in the queue waiting
to even start, so reporting it in the syslog is confusing,
especially since we already log the timeout via Connection::Poolable
This avoids a confusing sequence of error messages like the following:
[monitor(666)] node_timeout: 2 (elapsed: 2.00099802017212): GET http://127.0.0.1:7500/dev666/usage
[monitor(666)] Timeout contacting 127.0.0.1 dev 666 (http://127.0.0.1:7500/dev666/usage): took 2.25 seconds out of 2 allowed
Now, we only display the first message.
The timeout we're removing includes time spent in the queue waiting
to even start, so reporting it in the syslog is confusing,
especially since we already log the timeout via Connection::Poolable
This avoids a confusing sequence of error messages like the following:
[monitor(666)] node_timeout: 2 (elapsed: 2.00099802017212): GET http://127.0.0.1:7500/dev666/usage
[monitor(666)] Timeout contacting 127.0.0.1 dev 666 (http://127.0.0.1:7500/dev666/usage): took 2.25 seconds out of 2 allowed
Now, we only display the first message.
| lib/MogileFS/Worker/Monitor.pm | diffblobblamehistory |