Review Request 41048: set TaskStatus.slave_id field in TaskReconsiler

2015-12-07 Thread Tengfei Mu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41048/ --- Review request for Aurora. Repository: aurora Description --- set

Re: Review Request 40922: Thermos: Add ability to forward process output to stdout

2015-12-07 Thread Martin Hrabovcin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40922/ --- (Updated Dec. 7, 2015, 1:17 p.m.) Review request for Aurora. Bugs:

Re: Review Request 41048: set TaskStatus.slave_id field in TaskReconsiler

2015-12-07 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41048/#review109181 --- Master (676ee5b) is red with this patch.

Re: Review Request 41048: set TaskStatus.slave_id field in TaskReconsiler

2015-12-07 Thread Tengfei Mu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41048/ --- (Updated Dec. 7, 2015, 7:22 p.m.) Review request for Aurora and Maxim

Re: Review Request 41048: set TaskStatus.slave_id field in TaskReconsiler

2015-12-07 Thread Tengfei Mu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41048/ --- (Updated Dec. 7, 2015, 7:28 p.m.) Review request for Aurora and Maxim

Re: Review Request 40786: Replace manual Forwarding* with `@Forward`.

2015-12-07 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40786/#review109182 --- Ship it! This seems to be a minor change that we can reverse and

Re: Review Request 40786: Replace manual Forwarding* with `@Forward`.

2015-12-07 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40786/#review109183 --- I tried to commit this on master and applying the patch failed.

Re: Review Request 41048: set TaskStatus.slave_id field in TaskReconsiler

2015-12-07 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41048/#review109185 --- Ship it! Master (676ee5b) is green with this patch.

Re: Review Request 40786: Replace manual Forwarding* with `@Forward`.

2015-12-07 Thread John Sirois
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40786/ --- (Updated Dec. 7, 2015, 2:55 p.m.) Review request for Aurora, Bill Farner and

Re: Review Request 40786: Replace manual Forwarding* with `@Forward`.

2015-12-07 Thread John Sirois
> On Dec. 7, 2015, 12:29 p.m., Zameer Manji wrote: > > I tried to commit this on master and applying the patch failed. John, can > > you please rebase this? Should be all good now. - John --- This is an automatically generated e-mail.

Re: Review Request 40889: Changed mesos native lib to use mesos.executor instead

2015-12-07 Thread Zameer Manji
> On Dec. 2, 2015, 4:36 p.m., Maxim Khutornenko wrote: > > 3rdparty/python/BUILD, line 14 > > > > > > Am I reading this as revving up mesos native version in Aurora? If so, > > we should also update build.gradle and

Re: Review Request 40786: Replace manual Forwarding* with `@Forward`.

2015-12-07 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40786/#review109200 --- Ship it! Master (676ee5b) is green with this patch.

Re: Review Request 40889: Changed mesos native lib to use mesos.executor instead

2015-12-07 Thread Steve Niemitz
> On Dec. 3, 2015, 12:36 a.m., Maxim Khutornenko wrote: > > 3rdparty/python/BUILD, line 14 > > > > > > Am I reading this as revving up mesos native version in Aurora? If so, > > we should also update build.gradle

Re: Review Request 41056: Upgrade checkstyle and pmd versions.

2015-12-07 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41056/#review109215 --- Ship it! Ship It! - Zameer Manji On Dec. 7, 2015, 2:56 p.m.,

Re: Review Request 41056: Upgrade checkstyle and pmd versions.

2015-12-07 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41056/#review109219 --- Master (0c98e8a) is green with this patch.

Re: Review Request 40889: Changed mesos native lib to use mesos.executor instead

2015-12-07 Thread Steve Niemitz
> On Dec. 3, 2015, 12:36 a.m., Maxim Khutornenko wrote: > > 3rdparty/python/BUILD, line 14 > > > > > > Am I reading this as revving up mesos native version in Aurora? If so, > > we should also update build.gradle

Re: Review Request 41048: set TaskStatus.slave_id field in TaskReconsiler

2015-12-07 Thread Tengfei Mu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41048/ --- (Updated Dec. 8, 2015, 3:46 a.m.) Review request for Aurora and Maxim

Re: Review Request 41048: set TaskStatus.slave_id field in TaskReconsiler

2015-12-07 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41048/#review109271 --- Ship it! Master (d7a1619) is green with this patch.

Review Request 41074: Use lambdas throughout the project.

2015-12-07 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41074/ --- Review request for Aurora and Zameer Manji. Repository: aurora Description

Re: Review Request 41074: Use lambdas throughout the project.

2015-12-07 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41074/#review109279 --- Ship it! Master (d7a1619) is green with this patch.