mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Anand Mazumdar" <mazumdar.an...@gmail.com>
Subject Re: Review Request 41285: Initialized `pid` to None() instead of `UPID()` in Slave.
Date Tue, 12 Jan 2016 17:37:10 GMT

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

(Updated Jan. 12, 2016, 5:37 p.m.)


Review request for mesos, Ben Mahler, Jie Yu, and Vinod Kone.


Changes
-------

Modified summary.


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

Initialized `pid` to None() instead of `UPID()` in Slave.


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


Repository: mesos


Description
-------

This was earlier pointed out by Jie on https://reviews.apache.org/r/38874 . We never observed
it as we had no tests for recovery of HTTP executors/receiving status updates for them.


Diffs
-----

  src/slave/slave.cpp 90d0fecd2d83fd174134870a577ac59d79c0006f 

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


Testing
-------

make check


Thanks,

Anand Mazumdar


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