Re: Review Request 55476: Changed TASK_UNREACHABLE to be a non-terminal state.

2017-01-23 Thread Vinod Kone

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


Ship it!




Ship It!

- Vinod Kone


On Jan. 18, 2017, 7:33 p.m., Neil Conway wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/55476/
> ---
> 
> (Updated Jan. 18, 2017, 7:33 p.m.)
> 
> 
> Review request for mesos and Vinod Kone.
> 
> 
> Bugs: MESOS-6619
> https://issues.apache.org/jira/browse/MESOS-6619
> 
> 
> Repository: mesos
> 
> 
> Description
> ---
> 
> This task state was always conceptually non-terminal, but previously
> `isTerminalState` returned true for it. The master now distinguishes
> between "removable" and "terminal" task states, so we can correctly
> classify TASK_UNREACHABLE as a removable but non-terminal task state.
> 
> 
> Diffs
> -
> 
>   src/common/protobuf_utils.cpp af3c2a660f91c366ca5ffe8671d10c4a270bbbcc 
> 
> Diff: https://reviews.apache.org/r/55476/diff/
> 
> 
> Testing
> ---
> 
> `make check`
> 
> 
> Thanks,
> 
> Neil Conway
> 
>



Re: Review Request 55476: Changed TASK_UNREACHABLE to be a non-terminal state.

2017-01-18 Thread Neil Conway

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

(Updated Jan. 18, 2017, 7:33 p.m.)


Review request for mesos and Vinod Kone.


Changes
---

Tweak commit message.


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


Repository: mesos


Description (updated)
---

This task state was always conceptually non-terminal, but previously
`isTerminalState` returned true for it. The master now distinguishes
between "removable" and "terminal" task states, so we can correctly
classify TASK_UNREACHABLE as a removable but non-terminal task state.


Diffs (updated)
-

  src/common/protobuf_utils.cpp af3c2a660f91c366ca5ffe8671d10c4a270bbbcc 

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


Testing
---

`make check`


Thanks,

Neil Conway



Review Request 55476: Changed TASK_UNREACHABLE to be a non-terminal state.

2017-01-12 Thread Neil Conway

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

Review request for mesos and Vinod Kone.


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


Repository: mesos


Description
---

This task state was always conceptually non-terminal, but previously
`isTerminalState` returned true for it. Now that the master
distinguishes between "removable" and "terminal" task states, we can
correctly classify TASK_UNREACHABLE as a removal but non-terminal task
state.


Diffs
-

  src/common/protobuf_utils.cpp af3c2a660f91c366ca5ffe8671d10c4a270bbbcc 

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


Testing
---

`make check`


Thanks,

Neil Conway