> On Jan. 28, 2014, 5:08 a.m., Benjamin Hindman wrote:
> > src/slave/http.cpp, line 309
> > <https://reviews.apache.org/r/17443/diff/1/?file=452630#file452630line309>
> >
> >     Any reason not to use 'active_tasks' here instead of 'launched_tasks' 
> > too?

I went back and forth on the naming. This is my current thinking.

active_tasks on master = (queued_tasks + launched_tasks) on slaves.

calling them launched_tasks on the master seemed a bit confusing because it has 
a different meaning on the slave.

thoughts?


- Vinod


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


On Jan. 28, 2014, 3:02 a.m., Vinod Kone wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/17443/
> -----------------------------------------------------------
> 
> (Updated Jan. 28, 2014, 3:02 a.m.)
> 
> 
> Review request for mesos, Adam B, Benjamin Hindman, Ben Mahler, David 
> Robinson, and Niklas Nielsen.
> 
> 
> Bugs: MESOS-772
>     https://issues.apache.org/jira/browse/MESOS-772
> 
> 
> Repository: mesos-git
> 
> 
> Description
> -------
> 
> See summary.
> 
> 
> Diffs
> -----
> 
>   src/slave/http.cpp c8357e214d2adf2cd712072f58d07b07badb79dc 
> 
> Diff: https://reviews.apache.org/r/17443/diff/
> 
> 
> Testing
> -------
> 
> make
> 
> 
> Thanks,
> 
> Vinod Kone
> 
>

Reply via email to