mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Neil Conway <>
Subject Review Request 60162: Fixed bug in GroupTest.ConnectTimer, GroupTest.TimerCleanup.
Date Fri, 16 Jun 2017 18:26:10 GMT

This is an automatically generated e-mail. To reply, visit:

Review request for mesos and Andrei Budnik.

Repository: mesos


These tests looked for dispatches to `GroupProces::expired` as a way to
determine when the current Zk session has expired. However, the previous
implementation of the connect timer (`GroupProcess::timedout`) invoked
`GroupProcess::expired` directly, which meant that an `EXPECT_DISPATCH`
on the `expired` method should not fire.

However, a separate bug in `EXPECT_DISPATCH` (MESOS-5886) meant that the
test expectations were actually being satisfied by a dispatch to
`GroupProcess::timedout`, which meant the tests happened to work (!).

Fix this by changing `GroupProcess::timedout` to dispatch to `expired`
rather than invoking it directly.


  src/zookeeper/group.cpp 20f1928e05123d362147ce4ab0d6a752d95e466d 



`make check`

Validated that if the fix for MESOS-5886 is applied without this change, `GroupTest.ConnectTimer`
fails. With this change, the test passes (both with and without the fix for MESOS-5886 applied).


Neil Conway

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message