> On Oct. 15, 2014, 6:17 p.m., Niklas Nielsen wrote:
> > src/slave/status_update_manager.cpp, lines 292-293
> > <https://reviews.apache.org/r/26698/diff/1/?file=720967#file720967line292>
> >
> >     Can you use stream->next() instead ?

done.


- Vinod


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


On Oct. 14, 2014, 6 p.m., Vinod Kone wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/26698/
> -----------------------------------------------------------
> 
> (Updated Oct. 14, 2014, 6 p.m.)
> 
> 
> Review request for mesos, Adam B, Ben Mahler, and Niklas Nielsen.
> 
> 
> Repository: mesos-git
> 
> 
> Description
> -------
> 
> Added StatusUpdateManager::unacknowledged() API call so that slave can use 
> this info during reregistration.
> 
> 
> Diffs
> -----
> 
>   src/messages/messages.proto 8de7f9699f43aa2780f4a39fed087abcf5e5af99 
>   src/slave/status_update_manager.hpp 
> 24e3882ad1969c20a64daf90e408618c310e9a81 
>   src/slave/status_update_manager.cpp 
> 5d5cf234ef2dd47fa4b1f67be761dbca31659451 
>   src/tests/status_update_manager_tests.cpp 
> e9ef1e208cb01535e9366db7872b922c8f06ec40 
> 
> Diff: https://reviews.apache.org/r/26698/diff/
> 
> 
> Testing
> -------
> 
> make check
> 
> Ran new test 1000 times.
> 
> 
> Thanks,
> 
> Vinod Kone
> 
>

Reply via email to