You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@mesos.apache.org by Timothy Chen <tn...@apache.org> on 2016/04/01 00:29:09 UTC
Re: Review Request 45556: Minor style cleanups to docker health check
code.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45556/#review126478
-----------------------------------------------------------
Ship it!
Ship It!
- Timothy Chen
On March 31, 2016, 6:51 p.m., Anand Mazumdar wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/45556/
> -----------------------------------------------------------
>
> (Updated March 31, 2016, 6:51 p.m.)
>
>
> Review request for mesos, Jie Yu and Timothy Chen.
>
>
> Repository: mesos
>
>
> Description
> -------
>
> Minor cleanups around:
>
> - Remove periods at end of cerr/cout messages.
> - Line spacings.
> - A few of the `string` variables can be made `const`.
>
> This review mainly helps in reviewing r45557 easier.
>
>
> Diffs
> -----
>
> src/docker/executor.cpp afc769d0887e3842106e4c350e94c95c8ffc085e
>
> Diff: https://reviews.apache.org/r/45556/diff/
>
>
> Testing
> -------
>
> make check
>
>
> Thanks,
>
> Anand Mazumdar
>
>