mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Anand Mazumdar" <>
Subject Re: Review Request 40563: Added functionality for handling status updates from HTTP based executors
Date Sat, 21 Nov 2015 00:21:26 GMT

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

(Updated Nov. 21, 2015, 12:21 a.m.)

Review request for mesos, Ben Mahler, Isabel Jimenez, Marco Massenzio, and Vinod Kone.


Modified deps

Bugs: MESOS-3476

Repository: mesos


This change adds the ability for the agent to handle status updates from HTTP based executors.
Previously, the existing `statusUpdate` method used to handle status updates sent from slave
with `pid == UPID()`, sent from other executors/on behalf of other executors with `pid ==
Some()`. This change modifies the argument to be `Option<PID>`. This ensures that `pid
== None()` when set, the ACK is correctly routed to the corresponding HTTP based executor.


  src/slave/http.cpp ce48a0584ab18a8d95dd02619f62df18b2276639 
  src/slave/slave.hpp ec2dfa99e6b553e2bcd82d12db915ae8625075a1 
  src/slave/slave.cpp d1126f00d947fdb4823b0c495335b743254ac7ee 



make check + would add tests once the chain is reviewed.


Anand Mazumdar

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