mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Anand Mazumdar <>
Subject Re: Review Request 52144: Populated the timestamp field when sending status updates.
Date Sun, 25 Sep 2016 21:33:25 GMT

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

(Updated Sept. 25, 2016, 9:33 p.m.)

Review request for mesos and Vinod Kone.


Review comments, NNFR.

Repository: mesos


The command executor/default executor were not populating
the timestamp for the status update. Previously, the executor
driver used to do it underneath but that won't work with using
the HTTP command executor.

Diffs (updated)

  src/launcher/default_executor.cpp 2102fe8d70f0960fed669e1c4f0d6b6cd4af261c 
  src/launcher/executor.cpp e2ff1a88fd1c8a2045e390c84ccddc82f557de91 
  src/tests/default_executor_tests.cpp 786fbe3baac3c1542b318a26363213a4a1945ddd 



make check


Anand Mazumdar

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