----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52144/ -----------------------------------------------------------
(Updated Sept. 25, 2016, 9:33 p.m.) Review request for mesos and Vinod Kone. Changes ------- Review comments, NNFR. Repository: mesos Description ------- 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 Diff: https://reviews.apache.org/r/52144/diff/ Testing ------- make check Thanks, Anand Mazumdar
