Re: Review Request 49527: Reduce log level of finding a valid leader

2016-07-01 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49527/#review140418 --- Master (b3f88bc) is red with this patch.

Re: Review Request 49528: Patch thrift to compile under modern gcc.

2016-07-01 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49528/#review140417 --- Ship it! Master (b3f88bc) is green with this patch.

Re: Review Request 49527: Reduce log level of finding a valid leader

2016-07-01 Thread John Sirois
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49527/#review140415 --- Ship it! Ship It! - John Sirois On July 1, 2016, 1:29 p.m.,

Re: Review Request 49528: Patch thrift to compile under modern gcc.

2016-07-01 Thread John Sirois
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49528/#review140414 --- NB: With the `Makefile.in` patch, there are new WARNINGS

Re: Review Request 49528: Patch thrift to compile under modern gcc.

2016-07-01 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49528/#review140413 --- Ship it! Thanks! - Maxim Khutornenko On July 1, 2016, 7:28

Review Request 49527: Reduce log level of finding a valid leader

2016-07-01 Thread Stephan Erb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49527/ --- Review request for Aurora and John Sirois. Repository: aurora Description

Re: Review Request 49399: Fix Process log configuration handling.

2016-07-01 Thread Stephan Erb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49399/#review140408 --- Ship it! +1 for the bugfix as is. I also like your proposed

Re: Review Request 49399: Fix Process log configuration handling.

2016-07-01 Thread Stephan Erb
> On June 30, 2016, 3:39 p.m., Stephan Erb wrote: > > docs/reference/configuration.md, line 144 > > > > > > Have you considered using `RotatePolicy()` as the default rather than > > `Empty`? This could relieve us

Re: Review Request 49512: [FEEDBACK] Add thermos option to monitor whole docker storage disk usage

2016-07-01 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49512/#review140365 --- Master (b3f88bc) is red with this patch.

Review Request 49512: [FEEDBACK] Add thermos option to monitor whole docker storage disk usage

2016-07-01 Thread Martin Hrabovcin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49512/ --- Review request for Aurora. Repository: aurora Description --- I'd like