Re: Review Request 43112: Make --announcer-enable optional no-op instead of removing it completely.

2016-02-02 Thread Zhitao Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43112/ --- (Updated Feb. 2, 2016, 10:13 p.m.) Review request for Aurora, Kunal Thakar,

Re: Review Request 43112: Make --announcer-enable optional no-op instead of removing it completely.

2016-02-02 Thread Zhitao Li
> On Feb. 2, 2016, 10:08 p.m., Bill Farner wrote: > > Ship It! > > Bill Farner wrote: > Please also file a ticket to do the actual removal, make it a blocker for > https://issues.apache.org/jira/browse/AURORA-1586 https://issues.apache.org/jira/browse/AURORA-1606 - Zhitao

Re: Review Request 43112: Make --announcer-enable optional no-op instead of removing it completely.

2016-02-02 Thread Stephan Erb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43112/#review117502 --- Ship it! Ship It! - Stephan Erb On Feb. 2, 2016, 11:13

Re: Review Request 43112: Make --announcer-enable optional no-op instead of removing it completely.

2016-02-02 Thread Bill Farner
> On Feb. 2, 2016, 2:08 p.m., Bill Farner wrote: > > Ship It! Please also file a ticket to do the actual removal, make it a blocker for https://issues.apache.org/jira/browse/AURORA-1586 - Bill --- This is an automatically generated

Re: Review Request 43112: Make --announcer-enable optional no-op instead of removing it completely.

2016-02-02 Thread Kunal Thakar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43112/#review117504 --- Ship it! Ship It! - Kunal Thakar On Feb. 2, 2016, 10:13

Re: Review Request 43112: Make --announcer-enable optional no-op instead of removing it completely.

2016-02-02 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43112/#review117494 --- Ship it! Ship It! - Bill Farner On Feb. 2, 2016, 2:07 p.m.,

Re: Review Request 43112: Make --announcer-enable optional no-op instead of removing it completely.

2016-02-02 Thread John Sirois
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43112/#review117495 --- Please rebase against master to pickup NEWS reorganization. -

Re: Review Request 43112: Make --announcer-enable optional no-op instead of removing it completely.

2016-02-02 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43112/#review117520 --- This is on master now. - Zameer Manji On Feb. 2, 2016, 3:24

Re: Review Request 43112: Make --announcer-enable optional no-op instead of removing it completely.

2016-02-02 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43112/#review117507 --- Master (2f485e8) is green with this patch.

Re: Review Request 43112: Make --announcer-enable optional no-op instead of removing it completely.

2016-02-02 Thread Zhitao Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43112/ --- (Updated Feb. 2, 2016, 11:24 p.m.) Review request for Aurora, Kunal Thakar,

Re: Review Request 43112: Make --announcer-enable optional no-op instead of removing it completely.

2016-02-02 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43112/#review117505 ---

Re: Review Request 43112: Make --announcer-enable optional no-op instead of removing it completely.

2016-02-02 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43112/#review117519 --- Ship it! Ship It! - Zameer Manji On Feb. 2, 2016, 3:24