Re: Review Request 45701: Remove TaskQuery from killTasks RPC.

2016-04-06 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45701/#review127412 --- Ship it! Master (ddbb965) is green with this patch. ./build-s

Re: Review Request 45701: Remove TaskQuery from killTasks RPC.

2016-04-06 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45701/ --- (Updated April 6, 2016, 11:33 a.m.) Review request for Aurora, Maxim Khutornenk

Re: Review Request 45701: Remove TaskQuery from killTasks RPC.

2016-04-06 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45701/#review127399 --- Master (ddbb965) is red with this patch. ./build-support/jenkins

Re: Review Request 45701: Remove TaskQuery from killTasks RPC.

2016-04-06 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45701/ --- (Updated April 6, 2016, 10:37 a.m.) Review request for Aurora, Maxim Khutornenk

Re: Review Request 45701: Remove TaskQuery from killTasks RPC.

2016-04-06 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45701/#review127354 --- Ship it! Ship It! - Zameer Manji On April 4, 2016, 2:30 p.m.

Re: Review Request 45701: Remove TaskQuery from killTasks RPC.

2016-04-04 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45701/#review126955 --- Ship it! Ship It! - Maxim Khutornenko On April 4, 2016, 9:30

Re: Review Request 45701: Remove TaskQuery from killTasks RPC.

2016-04-04 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45701/#review126952 --- Ship it! Master (9847b02) is green with this patch. ./build-s

Re: Review Request 45701: Remove TaskQuery from killTasks RPC.

2016-04-04 Thread Bill Farner
> On April 4, 2016, 12:51 p.m., Maxim Khutornenko wrote: > > Does this test still pass with your change? > > https://github.com/apache/aurora/blob/master/src/test/python/apache/aurora/api_util.py#L88 All green now. - Bill --- This is a

Re: Review Request 45701: Remove TaskQuery from killTasks RPC.

2016-04-04 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45701/ --- (Updated April 4, 2016, 2:30 p.m.) Review request for Aurora, Maxim Khutornenko

Re: Review Request 45701: Remove TaskQuery from killTasks RPC.

2016-04-04 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45701/#review126924 --- Master (9847b02) is red with this patch. ./build-support/jenkins

Re: Review Request 45701: Remove TaskQuery from killTasks RPC.

2016-04-04 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45701/#review126921 --- Does this test still pass with your change? https://github.com/ap

Review Request 45701: Remove TaskQuery from killTasks RPC.

2016-04-04 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45701/ --- Review request for Aurora, Maxim Khutornenko and Zameer Manji. Bugs: AURORA-159