----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16038/ -----------------------------------------------------------
Review request for mesos, Benjamin Hindman, Brenden Matthews, and Vinod Kone. Repository: mesos-git Description ------- This was discovered in 0.16.0-rc1, https://reviews.apache.org/r/14434/ causes the master to raise an exception when there are no statuses available (pre-0.16.0 slave will not have any statuses when it re-registers with the master). I'm not convinced this is the only case where this can occur, since we do not always insert a status in Tasks, so I'm making this condition explicit to prevent this exception. We should take up a subsequent change to ensure that a status is always inserted for each state (we don't do this for STAGING in some places). Diffs ----- src/master/master.cpp 4f4db93a3c66d79e25783c7ea0617f5cf807c1aa src/slave/slave.cpp 75d9e5dfce9f546fb528d9f0fcf8ba127ac40b9c Diff: https://reviews.apache.org/r/16038/diff/ Testing ------- make check (I did not add a test here because the known case that triggers this occurs as the result of upgrading into 0.16.0) Thanks, Ben Mahler
