Re: Review Request 49322: Failed task status during EU is wrongly reported as SKIPPED_FAILED instead of TIMED_OUT

2016-06-28 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49322/#review139900 --- Ship it! Ship It! - Alejandro Fernandez On June 28, 2016,

Re: Review Request 49322: Failed task status during EU is wrongly reported as SKIPPED_FAILED instead of TIMED_OUT

2016-06-28 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49322/ --- (Updated June 28, 2016, 2:39 p.m.) Review request for Ambari, Alejandro

Re: Review Request 49322: Failed task status during EU is wrongly reported as SKIPPED_FAILED instead of TIMED_OUT

2016-06-28 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49322/#review139829 --- Ship it! Ship It! - Robert Levas On June 28, 2016, 11:08

Re: Review Request 49322: Failed task status during EU is wrongly reported as SKIPPED_FAILED instead of TIMED_OUT

2016-06-28 Thread Jonathan Hurley
> On June 28, 2016, 11:43 a.m., Nate Cole wrote: > > ambari-server/src/main/java/org/apache/ambari/server/actionmanager/ActionScheduler.java, > > lines 701-704 > > > > > > Could also use db.getTask(long), but this

Re: Review Request 49322: Failed task status during EU is wrongly reported as SKIPPED_FAILED instead of TIMED_OUT

2016-06-28 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49322/#review139807 --- Ship it!

Review Request 49322: Failed task status during EU is wrongly reported as SKIPPED_FAILED instead of TIMED_OUT

2016-06-28 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49322/ --- Review request for Ambari, Alejandro Fernandez, Nate Cole, and Robert Levas.