Re: Review Request 65294: Add revocable resources support to mesos-execute.

2018-01-26 Thread James Peach
> On Jan. 25, 2018, 9:10 p.m., Anish Gupta wrote: > > src/cli/execute.cpp > > Lines 1077-1082 (patched) > > > > > > Why not unconditionally set it since resources are not revocable unless > > have RevocableInfo >

Re: Review Request 65294: Add revocable resources support to mesos-execute.

2018-01-26 Thread Zhitao Li
> On Jan. 25, 2018, 9:10 p.m., Anish Gupta wrote: > > src/cli/execute.cpp > > Lines 1077-1082 (patched) > > > > > > Why not unconditionally set it since resources are not revocable unless > > have RevocableInfo I

Re: Review Request 65294: Add revocable resources support to mesos-execute.

2018-01-25 Thread Anish Gupta
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65294/#review196286 --- src/cli/execute.cpp Lines 1077-1082 (patched)

Re: Review Request 65294: Add revocable resources support to mesos-execute.

2018-01-23 Thread Mesos Reviewbot Windows
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65294/#review196053 --- PASS: Mesos patch 65294 was successfully built and tested. Review

Review Request 65294: Add revocable resources support to mesos-execute.

2018-01-23 Thread Zhitao Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65294/ --- Review request for mesos and Jason Lai. Bugs: MESOS-8471 https://issues.apa