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

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

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

2015-11-30 Thread Bill Farner
> On Nov. 24, 2015, 1:15 p.m., Zameer Manji wrote: > > LGTM assuming E2E tests pass and 0.10.0 client can communicate against this > > scheduler. Confirmed. E2E tests pass with scheduler/client individually rolled forward. - Bill ---

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.

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 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 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