Re: Review Request 56523: Displaying update id after 'Killed for job update' message for the update that resulted in the task getting killed

2017-02-13 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56523/#review165387 --- This is now on master. - Zameer Manji On Feb. 13, 2017, 12:09 p

Re: Review Request 56523: Displaying update id after 'Killed for job update' message for the update that resulted in the task getting killed

2017-02-13 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56523/#review165385 --- Ship it! LGTM, thanks for your contribution. - Zameer Manji

Re: Review Request 56523: Displaying update id after 'Killed for job update' message for the update that resulted in the task getting killed

2017-02-13 Thread Abhishek Jain
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56523/ --- (Updated Feb. 13, 2017, 12:09 p.m.) Review request for Aurora, Stephan Erb and

Re: Review Request 56523: Displaying update id after 'Killed for job update' message for the update that resulted in the task getting killed

2017-02-12 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56523/#review165281 --- Ship it! Master (ad3377a) is green with this patch. ./build-s

Re: Review Request 56523: Displaying update id after 'Killed for job update' message for the update that resulted in the task getting killed

2017-02-12 Thread Stephan Erb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56523/#review165280 --- Ship it! Ship It! - Stephan Erb On Feb. 10, 2017, 8:24 p.m.,

Re: Review Request 56523: Displaying update id after 'Killed for job update' message for the update that resulted in the task getting killed

2017-02-12 Thread Stephan Erb
> On Feb. 12, 2017, 10:16 p.m., Aurora ReviewBot wrote: > > Master (ad3377a) is red with this patch. > > ./build-support/jenkins/build.sh > > > > Test coverage missing for org/apache/aurora/scheduler/base/AsyncUtil$2 > > Test coverage missing for org/apache/aurora/Protobufs > > Test cove

Re: Review Request 56523: Displaying update id after 'Killed for job update' message for the update that resulted in the task getting killed

2017-02-12 Thread Stephan Erb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56523/#review165279 --- @ReviewBot retry - Stephan Erb On Feb. 10, 2017, 8:24 p.m., Abh

Re: Review Request 56523: Displaying update id after 'Killed for job update' message for the update that resulted in the task getting killed

2017-02-12 Thread Abhishek Jain
> On Feb. 12, 2017, 9:16 p.m., Aurora ReviewBot wrote: > > Master (ad3377a) is red with this patch. > > ./build-support/jenkins/build.sh > > > > Test coverage missing for org/apache/aurora/scheduler/base/AsyncUtil$2 > > Test coverage missing for org/apache/aurora/Protobufs > > Test cover

Re: Review Request 56523: Displaying update id after 'Killed for job update' message for the update that resulted in the task getting killed

2017-02-12 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56523/#review165275 --- Master (ad3377a) is red with this patch. ./build-support/jenkins

Re: Review Request 56523: Displaying update id after 'Killed for job update' message for the update that resulted in the task getting killed

2017-02-12 Thread Stephan Erb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56523/#review165274 --- @ReviewBot retry - Stephan Erb On Feb. 10, 2017, 8:24 p.m., Abh

Re: Review Request 56523: Displaying update id after 'Killed for job update' message for the update that resulted in the task getting killed

2017-02-10 Thread Abhishek Jain
> On Feb. 10, 2017, 7:36 p.m., Aurora ReviewBot wrote: > > Master (ad3377a) is red with this patch. > > ./build-support/jenkins/build.sh > > > > Test coverage missing for > > org/apache/aurora/scheduler/discovery/ServiceDiscoveryModule > > Test coverage missing for > > org/apache/aurora/

Re: Review Request 56523: Displaying update id after 'Killed for job update' message for the update that resulted in the task getting killed

2017-02-10 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56523/#review165152 --- Master (ad3377a) is red with this patch. ./build-support/jenkins

Re: Review Request 56523: Displaying update id after 'Killed for job update' message for the update that resulted in the task getting killed

2017-02-10 Thread Abhishek Jain
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56523/ --- (Updated Feb. 10, 2017, 7:24 p.m.) Review request for Aurora, Stephan Erb and Z

Re: Review Request 56523: Displaying update id after 'Killed for job update' message for the update that resulted in the task getting killed

2017-02-10 Thread Stephan Erb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56523/#review165134 --- This looks very useful. Thanks! src/main/java/org/apache/aurora/

Re: Review Request 56523: Displaying update id after 'Killed for job update' message for the update that resulted in the task getting killed

2017-02-09 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56523/#review165047 --- Ship it! Master (ad3377a) is green with this patch. ./build-s

Review Request 56523: Displaying update id after 'Killed for job update' message for the update that resulted in the task getting killed

2017-02-09 Thread Abhishek Jain
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56523/ --- Review request for Aurora and Zameer Manji. Repository: aurora Description --