More logging in HHWheelTimer::scheduleTimeoutFn
authorHans Fugal <fugalh@fb.com>
Fri, 5 Dec 2014 17:46:56 +0000 (09:46 -0800)
committerDave Watson <davejwatson@fb.com>
Thu, 11 Dec 2014 16:00:58 +0000 (08:00 -0800)
Summary:
This was supposed to be an update to D1720014 but I screwed up with arc.

Test Plan: run the test by hand and see the output

Reviewed By: jsedgwick@fb.com

Subscribers: exa, njormrod, folly-diffs@

FB internal diff: D1721951

Signature: t1:1721951:1417801009:72a27ec2ca473a996785ff9cae48bd51a43d9045

folly/io/async/HHWheelTimer.h

index eff837e5a645091525f78a43fafd75ff7497d50a..062544b5ef16738958796ca334095f93173e0ce5 100644 (file)
@@ -185,8 +185,11 @@ class HHWheelTimer : private folly::AsyncTimeout,
         try {
           fn_();
         } catch (std::exception const& e) {
-          LOG(ERROR) << e.what();
-        } catch (...) { }
+          LOG(ERROR) << "HHWheelTimer timeout callback threw an exception: "
+            << e.what();
+        } catch (...) {
+          LOG(ERROR) << "HHWheelTimer timeout callback threw a non-exception.";
+        }
         delete this;
       }
       F fn_;