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


Thanks Alex, didn't realize I was on the reviewers here, so apologies for the 
delay! Why don't we just name the constant with DEFAULT to avoid the need for 
this and to make the meaning of the constant more clear?


src/tests/fault_tolerance_tests.cpp (lines 620 - 621)
<https://reviews.apache.org/r/38161/#comment158295>

    Don't think you need these comments, we should just 
s/REGISTRATION_BACKOFF_FACTOR/DEFAULT_REGISTRATION_BACKOFF_FACTOR/ to be more 
explicit about it. We've started doing this for some of our constants already, 
but need to continue! :)



src/tests/fault_tolerance_tests.cpp (lines 1547 - 1551)
<https://reviews.apache.org/r/38161/#comment158296>

    Ditto here, it seems like if we use DEFAULT in the name, we don't have to 
bother setting it here, thoughts?


- Ben Mahler


On Sept. 28, 2015, 5:08 p.m., Alexander Rukletsov wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/38161/
> -----------------------------------------------------------
> 
> (Updated Sept. 28, 2015, 5:08 p.m.)
> 
> 
> Review request for mesos and Ben Mahler.
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> See summary.
> 
> 
> Diffs
> -----
> 
>   src/tests/fault_tolerance_tests.cpp 
> c97bc4691f9bac4a8677e6d2247be96ee9674b57 
> 
> Diff: https://reviews.apache.org/r/38161/diff/
> 
> 
> Testing
> -------
> 
> make check (Mac OS 10.10.4)
> 
> 
> Thanks,
> 
> Alexander Rukletsov
> 
>

Reply via email to