Re: Review Request 32541: Adding client Kerberos support.

2015-04-01 Thread Maxim Khutornenko
On April 1, 2015, 8:40 p.m., Kevin Sweeney wrote: src/main/python/apache/aurora/client/cli/BUILD, line 29 https://reviews.apache.org/r/32541/diff/2/?file=909509#file909509line29 How about kaurora This is a joke, right? :) If not I am really struggling how it would be more

Re: Review Request 32329: Extract job key from RPC parameters

2015-04-01 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32329/#review78551 --- Ship it!

Re: Review Request 32352: Making preemptor asynchronous. Part 3(final) - background service.

2015-04-01 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32352/#review78603 --- Ship it! Ship It! - Bill Farner On March 21, 2015, 2:19 a.m.,

Re: Review Request 32541: Adding client Kerberos support.

2015-04-01 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32541/#review78573 --- Ship it! Looking good modulo comments below. Have you had a chance

Re: Review Request 32541: Adding client Kerberos support.

2015-04-01 Thread Maxim Khutornenko
On March 31, 2015, 7:30 p.m., Kevin Sweeney wrote: src/main/python/apache/aurora/common/auth/__init__.py, line 14 https://reviews.apache.org/r/32541/diff/2/?file=909512#file909512line14 I see red here but I don't see any corresponding changes to import lines - were these