> On Aug. 28, 2013, 2:17 a.m., Vinson Lee wrote:
> > I see the same build failure on Fedora 18 and confirm the build failure is 
> > also introduced with commit 400a88f9817bb102522b08b83dca400380ad8a9b.
> > 
> > commit 400a88f9817bb102522b08b83dca400380ad8a9b
> > Author: Vinod Kone <[email protected]>
> > Date:   Tue Aug 27 13:36:42 2013 -0400
> > 
> >     Added resource checks to slave recovery tests to ensure resources
> >     are re-offered after restarts.
> >     
> >     From: Ian Downes <[email protected]>
> >     Review: https://reviews.apache.org/r/13764
> > 
> > 
> > tests/slave_recovery_tests.cpp: In member function ‘virtual void 
> > SlaveRecoveryTest_KillTask_Test<gtest_TypeParam_>::TestBody()’:
> > tests/slave_recovery_tests.cpp:1289:26: error: redeclaration of 
> > ‘process::Future<std::vector<mesos::Offer> > offers2’
> > tests/slave_recovery_tests.cpp:1265:26: error: 
> > ‘process::Future<std::vector<mesos::Offer> > offers2’ previously declared 
> > here
> >

this fix is incomplete, please see https://reviews.apache.org/r/13881/


- Ian


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


On Aug. 28, 2013, 12:19 a.m., Chi Zhang wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13876/
> -----------------------------------------------------------
> 
> (Updated Aug. 28, 2013, 12:19 a.m.)
> 
> 
> Review request for mesos, Benjamin Hindman, Ben Mahler, Ian Downes, and Vinod 
> Kone.
> 
> 
> Repository: mesos-git
> 
> 
> Description
> -------
> 
> commit 400a88f9817bb102522b08b83dca400380ad8a9b introduced a build error where
>     `offers2' is declared twice. Remove the latter one.
>  
> 
> 
> Diffs
> -----
> 
>   src/tests/slave_recovery_tests.cpp 78f42ff 
> 
> Diff: https://reviews.apache.org/r/13876/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Chi Zhang
> 
>

Reply via email to