mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Benjamin Bannier <>
Subject Re: Review Request 54664: Acked correct task id in DefaultExecutorTest.KillTaskGroupOnTaskFailure.
Date Tue, 13 Dec 2016 11:50:14 GMT

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

(Updated Dec. 13, 2016, 12:50 p.m.)

Review request for mesos, Anand Mazumdar and Vinod Kone.


Sprinkle some vertical space goodness.

Bugs: MESOS-6744

Repository: mesos

Description (updated)

When acknowledging task status updates, this test was erroneously
using a cached task id instead of the actual task id send in the
status update. This can lead to failed acknowledgements when
status updates appear in different order.

This patch just uses the task id sent as part of the status update in
the acknowledgement.

Diffs (updated)

  src/tests/default_executor_tests.cpp a88796b83c17fb01e7698907e9b0899a63700782 



Test did not fail in my setup in 4000 iterations.


Benjamin Bannier

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