mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Mesos ReviewBot <revi...@mesos.apache.org>
Subject Re: Review Request 52231: Fixed driver based schedulers to ACK updates from HTTP executors.
Date Sat, 24 Sep 2016 19:14:05 GMT

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



Bad patch!

Reviews applied: [52231]

Failed command: ./support/apply-review.sh -n -r 52231

Error:
2016-09-24 19:14:36 URL:https://reviews.apache.org/r/52231/diff/raw/ [4647/4647] -> "52231.patch"
[1]
error: patch failed: src/tests/command_executor_tests.cpp:201
error: src/tests/command_executor_tests.cpp: patch does not apply

Full log: https://builds.apache.org/job/mesos-reviewbot/15380/console

- Mesos ReviewBot


On Sept. 24, 2016, 1:07 a.m., Anand Mazumdar wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/52231/
> -----------------------------------------------------------
> 
> (Updated Sept. 24, 2016, 1:07 a.m.)
> 
> 
> Review request for mesos and Benjamin Mahler.
> 
> 
> Bugs: MESOS-6245
>     https://issues.apache.org/jira/browse/MESOS-6245
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> Previously, driver based schedulers were not able to acknowledge
> status updates from HTTP based executors if they had explicit
> acknowledgements enabled. This was due to the fact that we
> were not populating `TaskStatus.slave_id` correctly if not
> set. It would also be great to validate `TaskStatus.slave_id`
> set by the executor and let them know if the value is incorrect.
> 
> 
> Diffs
> -----
> 
>   src/common/protobuf_utils.cpp 5db4be4bdc7b9a3a2a66a17f8a9ac74c8d3dfbf6 
>   src/slave/slave.cpp 11e9c8af87aa5153f72f2a20cc578fe3d729b153 
>   src/tests/command_executor_tests.cpp 07e5eb4d7c2ace2b6714fbe02f29d41663152556 
> 
> Diff: https://reviews.apache.org/r/52231/diff/
> 
> 
> Testing
> -------
> 
> make check
> 
> 
> Thanks,
> 
> Anand Mazumdar
> 
>


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