> On Oct. 10, 2014, 11:26 p.m., Till Toenshoff wrote:
> > src/slave/main.cpp, line 117
> > <https://reviews.apache.org/r/26509/diff/5/?file=717907#file717907line117>
> >
> >     Not yours but this would trigger a glog warning as it has not yet been 
> > initialized. We should possibly check all runnables for such nits at some  
> > point.

I was actually wrong by claiming that this would trigger a glog warning -- 
doublechecked it, turns out that it doesnt.


- Till


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


On Oct. 11, 2014, 1:25 a.m., Kapil Arya wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/26509/
> -----------------------------------------------------------
> 
> (Updated Oct. 11, 2014, 1:25 a.m.)
> 
> 
> Review request for mesos, Niklas Nielsen and Till Toenshoff.
> 
> 
> Repository: mesos-git
> 
> 
> Description
> -------
> 
> Added --module flag for Mesos slave.
> 
> 
> Diffs
> -----
> 
>   src/slave/flags.hpp 16f0cc2ab5ba16a39499608174278b3082e0585d 
>   src/slave/main.cpp 1eafb356b58613d4ab8bd6dd245583e0d6f25a97 
> 
> Diff: https://reviews.apache.org/r/26509/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Kapil Arya
> 
>

Reply via email to