Review Request 40680: Add deps to centos 7 image to handle thrift patch.

2015-11-24 Thread John Sirois
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40680/ --- Review request for Aurora, Bill Farner and Zameer Manji. Repository:

Review Request 40675: Kill extraneous `make clean`.

2015-11-24 Thread John Sirois
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40675/ --- Review request for Aurora and Bill Farner. Repository: aurora Description

Re: Review Request 40680: Add deps to centos 7 image to handle thrift patch.

2015-11-24 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40680/#review107880 --- Ship it! Ship It! - Bill Farner On Nov. 24, 2015, 3:51 p.m.,

Review Request 40673: Set JDK and langauge level for entire project.

2015-11-24 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40673/ --- Review request for Aurora and Bill Farner. Repository: aurora Description

Re: Review Request 40673: Set JDK and langauge level for entire project.

2015-11-24 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40673/#review107855 --- Ship it! Ship It! - Bill Farner On Nov. 24, 2015, 1:29 p.m.,

Re: Review Request 40656: Remove SessionKey from APIs and implementations.

2015-11-24 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40656/ --- (Updated Nov. 24, 2015, 12:29 p.m.) Review request for Aurora, Maxim

Re: Review Request 40656: Remove SessionKey from APIs and implementations.

2015-11-24 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40656/#review107846 --- Ship it! LGTM assuming E2E tests pass and 0.10.0 client can

Re: Review Request 40656: Remove SessionKey from APIs and implementations.

2015-11-24 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40656/#review107850 --- Ship it! Master (697087b) is green with this patch.

Review Request 40689: Upgrade RBTools to 0.7.5

2015-11-24 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40689/ --- Review request for Aurora, John Sirois and Bill Farner. Repository: aurora

Re: Review Request 40689: Upgrade RBTools to 0.7.5

2015-11-24 Thread John Sirois
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40689/#review107895 --- Ship it! Ship It! - John Sirois On Nov. 24, 2015, 7 p.m.,

Re: Review Request 40689: Upgrade RBTools to 0.7.5

2015-11-24 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40689/#review107896 --- Ship it! Ship It! - Bill Farner On Nov. 24, 2015, 6 p.m.,

Re: Review Request 40689: Upgrade RBTools to 0.7.5

2015-11-24 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40689/#review107898 --- Ship it! Master (e3c5c37) is green with this patch.

Re: Review Request 40680: Add deps to centos 7 image to handle thrift patch.

2015-11-24 Thread John Sirois
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40680/ --- (Updated Nov. 24, 2015, 7:54 p.m.) Review request for Aurora, Bill Farner and

Re: Review Request 40680: Add deps to centos 7 image to handle thrift patch.

2015-11-24 Thread John Sirois
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40680/ --- (Updated Nov. 24, 2015, 8:06 p.m.) Review request for Aurora, Bill Farner and

Re: Review Request 40675: Kill extraneous `make clean`.

2015-11-24 Thread John Sirois
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40675/ --- (Updated Nov. 24, 2015, 8:06 p.m.) Review request for Aurora and Bill Farner.

Review Request 40658: Copy artifacts from containers rather than using volume mounts.

2015-11-24 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40658/ --- Review request for Aurora and Zameer Manji. Bugs: AURORA-1539

Re: Review Request 40656: Remove SessionKey from APIs and implementations.

2015-11-24 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40656/#review107806 --- Master (cd4f7ad) is red with this patch.

Review Request 40662: Remove jersey dependency from commons code.

2015-11-24 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40662/ --- Review request for Aurora and Zameer Manji. Repository: aurora Description

Re: Review Request 40662: Remove jersey dependency from commons code.

2015-11-24 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40662/#review107815 --- Ship it! Ship It! - Zameer Manji On Nov. 24, 2015, 10:41 a.m.,

Re: Review Request 40662: Remove jersey dependency from commons code.

2015-11-24 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40662/#review107816 --- Ship it! Master (cd4f7ad) is green with this patch.

Review Request 40656: Remove SessionKey from APIs and implementations.

2015-11-24 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40656/ --- Review request for Aurora, Maxim Khutornenko and Zameer Manji. Bugs:

Re: Review Request 40658: Copy artifacts from containers rather than using volume mounts.

2015-11-24 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40658/#review107821 --- Ship it! Ship It! - Zameer Manji On Nov. 24, 2015, 9:24 a.m.,

Re: Review Request 40656: Remove SessionKey from APIs and implementations.

2015-11-24 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40656/#review107822 --- api/src/main/thrift/org/apache/aurora/gen/api.thrift

Review Request 40666: Remove unnecessary volume mount argument.

2015-11-24 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40666/ --- Review request for Aurora and Zameer Manji. Repository: aurora-packaging

Re: Review Request 40666: Remove unnecessary volume mount argument.

2015-11-24 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40666/#review107827 --- Ship it! Ship It! - Zameer Manji On Nov. 24, 2015, 11:36 a.m.,