Re: Review Request 35910: Updated the executor driver to set TaskStatus.uuid.

2015-06-26 Thread Michael Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35910/#review89565 --- Just a question: what do we mean by a pure client here? - Michael

Review Request 35910: Updated the executor driver to set TaskStatus.uuid.

2015-06-26 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35910/ --- Review request for mesos and Vinod Kone. Repository: mesos Description

Re: Review Request 35910: Updated the executor driver to set TaskStatus.uuid.

2015-06-26 Thread Ben Mahler
On June 26, 2015, 9:23 p.m., Michael Park wrote: Just a question: what do we mean by a pure client here? Before we figured out that HTTP API was a better name, we used to call it pure as in the clients don't need libmesos pulled in. Removed the reference to pure here :) - Ben

Re: Review Request 35910: Updated the executor driver to set TaskStatus.uuid.

2015-06-26 Thread Michael Park
On June 26, 2015, 9:23 p.m., Michael Park wrote: Just a question: what do we mean by a pure client here? Ben Mahler wrote: Before we figured out that HTTP API was a better name, we used to call it pure as in the clients don't need libmesos pulled in. Removed the reference to pure

Re: Review Request 35910: Updated the executor driver to set TaskStatus.uuid.

2015-06-26 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35910/#review89573 --- src/exec/exec.cpp (line 492)

Re: Review Request 35910: Updated the executor driver to set TaskStatus.uuid.

2015-06-26 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35910/ --- (Updated June 27, 2015, 12:36 a.m.) Review request for mesos and Vinod Kone.

Re: Review Request 35910: Updated the executor driver to set TaskStatus.uuid.

2015-06-26 Thread Ben Mahler
On June 26, 2015, 9:44 p.m., Vinod Kone wrote: src/exec/exec.cpp, lines 496-497 https://reviews.apache.org/r/35910/diff/1/?file=993586#file993586line496 s/create/overwrite/ s/pure clients/HTTP API/ Can you move this down to #514 where we also overwrite

Re: Review Request 35910: Updated the executor driver to set TaskStatus.uuid.

2015-06-26 Thread Vinod Kone
On June 26, 2015, 9:44 p.m., Vinod Kone wrote: src/exec/exec.cpp, lines 496-497 https://reviews.apache.org/r/35910/diff/1/?file=993586#file993586line496 s/create/overwrite/ s/pure clients/HTTP API/ Can you move this down to #514 where we also overwrite