mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Marco Massenzio" <ma...@mesosphere.io>
Subject Re: Review Request 33490: Avoid adding duplicate tasks
Date Fri, 08 May 2015 18:14:15 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/33490/
-----------------------------------------------------------

(Updated May 8, 2015, 6:14 p.m.)


Review request for mesos, Adam B and Joris Van Remoortere.


Summary (updated)
-----------------

Avoid adding duplicate tasks


Bugs: MESOS-2633
    https://issues.apache.org/jira/browse/MESOS-2633


Repository: mesos


Description
-------

In Framework::addCompletedTask(const Task& task) we did not check
for duplicated tasks, so they could be added more than once.

A simple check has now been added (there still is the issue
of whether the `operator ==` on `Task` is too strict, so as
never to cause a match).


Diffs
-----

  src/master/framework.cpp PRE-CREATION 

Diff: https://reviews.apache.org/r/33490/diff/


Testing
-------

buildbot make check pass
http://build.mesosphere.com:5555/builders/dev_test/builds/13


Thanks,

Marco Massenzio


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