Re: Review Request 60714: aurora job restart request should be idempotent and retryable

2017-07-17 Thread Santhosh Kumar Shanmugham
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60714/#review180756 --- Ship it! Ship It! - Santhosh Kumar Shanmugham On July 15,

Re: Review Request 60714: aurora job restart request should be idempotent and retryable

2017-07-16 Thread Mehrdad Nurolahzade
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60714/#review180646 --- Ship it! Ship It! - Mehrdad Nurolahzade On July 15, 2017,

Re: Review Request 60714: aurora job restart request should be idempotent and retryable

2017-07-16 Thread Kai Huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60714/ --- (Updated July 16, 2017, 6 a.m.) Review request for Aurora, David McLaughlin,

Re: Review Request 60714: aurora job restart request should be idempotent and retryable

2017-07-15 Thread David McLaughlin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60714/#review180637 --- Ship it! Ship It! - David McLaughlin On July 7, 2017, 11:52

Re: Review Request 60714: aurora job restart request should be idempotent and retryable

2017-07-14 Thread Mehrdad Nurolahzade
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60714/#review180632 --- Looks legit. Just as Santhosh said let's make sure semantics of

Re: Review Request 60714: aurora job restart request should be idempotent and retryable

2017-07-13 Thread Reza Motamedi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60714/#review180444 --- Ship it! LGTM. - Reza Motamedi On July 7, 2017, 11:52 p.m.,

Re: Review Request 60714: aurora job restart request should be idempotent and retryable

2017-07-12 Thread Santhosh Kumar Shanmugham
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60714/#review180318 --- LGTM. Can you test to make sure that job create and job add work

Re: Review Request 60714: aurora job restart request should be idempotent and retryable

2017-07-07 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60714/#review179972 --- Ship it! Master (a922b05) is green with this patch.

Re: Review Request 60714: aurora job restart request should be idempotent and retryable

2017-07-07 Thread Kai Huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60714/ --- (Updated July 7, 2017, 11:52 p.m.) Review request for Aurora, David

Re: Review Request 60714: aurora job restart request should be idempotent and retryable

2017-07-07 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60714/#review179935 --- Ship it! Master (a922b05) is green with this patch.

Re: Review Request 60714: aurora job restart request should be idempotent and retryable

2017-07-07 Thread Kai Huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60714/ --- (Updated July 7, 2017, 8:20 p.m.) Review request for Aurora, David McLaughlin,

Review Request 60714: aurora job restart request should be idempotent and retryable

2017-07-07 Thread Kai Huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60714/ --- Review request for Aurora, David McLaughlin, Mehrdad Nurolahzade, Santhosh