Re: Review Request 31652: Returning pending reason for all tasks in a group

2015-03-05 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31652/#review75413 --- Ship it! Master (70494a1) is green with this patch.

Re: Review Request 31652: Returning pending reason for all tasks in a group

2015-03-05 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31652/ --- (Updated March 5, 2015, 6:48 p.m.) Review request for Aurora, Bill Farner and

Re: Review Request 31652: Returning pending reason for all tasks in a group

2015-03-05 Thread Maxim Khutornenko
On March 3, 2015, 11:56 p.m., Bill Farner wrote: src/main/java/org/apache/aurora/scheduler/events/PubsubEvent.java, line 237 https://reviews.apache.org/r/31652/diff/1/?file=882474#file882474line237 s/taskId/groupKey/ Good catch, fixed. On March 3, 2015, 11:56 p.m., Bill Farner

Re: Review Request 31652: Returning pending reason for all tasks in a group

2015-03-03 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31652/#review75096 --- Ship it! Nice!

Re: Review Request 31652: Returning pending reason for all tasks in a group

2015-03-03 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31652/#review75099 --- Ship it! Ship It! - Zameer Manji On March 2, 2015, 4:58 p.m.,

Re: Review Request 31652: Returning pending reason for all tasks in a group

2015-03-02 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31652/ --- (Updated March 3, 2015, 12:58 a.m.) Review request for Aurora, Bill Farner and

Re: Review Request 31652: Returning pending reason for all tasks in a group

2015-03-02 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31652/#review74865 --- Master (4cca6a6) is red with this patch.