You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@mesos.apache.org by Anand Mazumdar <ma...@gmail.com> on 2016/03/18 04:27:24 UTC
Review Request 45013: Removed un-needed methods from command
scheduler.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45013/
-----------------------------------------------------------
Review request for mesos and Vinod Kone.
Bugs: MESOS-3559
https://issues.apache.org/jira/browse/MESOS-3559
Repository: mesos
Description
-------
This change cleans up the unused scheduler driver methods
that would not be needed after moving to the scheduler library.
This review mainly eases reviewing later in the chain and can't
be committed separately.
Diffs
-----
src/cli/execute.cpp ed42cb568f3d16856f48b3bbd354cb2b0fb83e8e
Diff: https://reviews.apache.org/r/45013/diff/
Testing
-------
This change should not be committed with the review after it.
Thanks,
Anand Mazumdar
Re: Review Request 45013: Removed un-needed methods from command
scheduler.
Posted by Vinod Kone <vi...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45013/#review125750
-----------------------------------------------------------
Ship it!
Ship It!
- Vinod Kone
On March 18, 2016, 3:27 a.m., Anand Mazumdar wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/45013/
> -----------------------------------------------------------
>
> (Updated March 18, 2016, 3:27 a.m.)
>
>
> Review request for mesos and Vinod Kone.
>
>
> Bugs: MESOS-3559
> https://issues.apache.org/jira/browse/MESOS-3559
>
>
> Repository: mesos
>
>
> Description
> -------
>
> This change cleans up the unused scheduler driver methods
> that would not be needed after moving to the scheduler library.
>
> This review mainly eases reviewing later in the chain and can't
> be committed separately.
>
>
> Diffs
> -----
>
> src/cli/execute.cpp ed42cb568f3d16856f48b3bbd354cb2b0fb83e8e
>
> Diff: https://reviews.apache.org/r/45013/diff/
>
>
> Testing
> -------
>
> This change should not be committed with the review after it.
>
>
> Thanks,
>
> Anand Mazumdar
>
>