Re: Review Request 50819: Use update_job instead of creating new config object when modifying.

2016-08-05 Thread Stephan Erb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50819/#review144955 --- Given that this sneaked past our test coverage, do you see an

Re: Review Request 50859: Update install docs for 0.15.0

2016-08-05 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50859/#review144956 --- Ship it! Master (ec3591c) is green with this patch.

Re: Review Request 50859: Update install docs for 0.15.0

2016-08-05 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50859/#review144951 --- Ship it! Ship It! - Maxim Khutornenko On Aug. 5, 2016, 4:52

Review Request 50859: Update install docs for 0.15.0

2016-08-05 Thread Stephan Erb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50859/ --- Review request for Aurora and Maxim Khutornenko. Repository: aurora

Re: Review Request 50826: Populate the source field of ExecutorInfo.

2016-08-05 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50826/ --- (Updated Aug. 5, 2016, 10:50 a.m.) Review request for Aurora, Joshua Cohen and

Re: Review Request 50685: Support TBinaryProtocol over HTTP

2016-08-05 Thread Zameer Manji
> On Aug. 4, 2016, 11 a.m., Kunal Thakar wrote: > > RELEASE-NOTES.md, line 20 > > > > > > s/with/without Thanks for pointing this out. Fixed in

Re: Review Request 50826: Populate the source field of ExecutorInfo.

2016-08-05 Thread Joshua Cohen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50826/#review144930 --- Ship it! Looks good to me. Would you mind adding something to