Re: Review Request 57147: Added support for processing custom command script timeout

2017-03-02 Thread Alejandro Fernandez
> On March 1, 2017, 7:07 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/metainfo.xml > > Lines 64 (patched) > > > > > > I'm still opposed to this patch fundamental

Re: Review Request 57147: Added support for processing custom command script timeout

2017-03-02 Thread Laszlo Puskas
> On March 1, 2017, 7:07 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/metainfo.xml > > Lines 64 (patched) > > > > > > I'm still opposed to this patch fundamental

Re: Review Request 57147: Added support for processing custom command script timeout

2017-03-01 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57147/#review167547 --- ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/m

Re: Review Request 57147: Added support for processing custom command script timeout

2017-03-01 Thread Laszlo Puskas
> On Feb. 28, 2017, 6:32 p.m., Alejandro Fernandez wrote: > > Thanks Alejanro, i wasn't ware of that; I'll do the modification to the other approach then. > On Feb. 28, 2017, 6:32 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/metainfo.xml

Re: Review Request 57147: Added support for processing custom command script timeout

2017-02-28 Thread Alejandro Fernandez
> On Feb. 28, 2017, 6:32 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/metainfo.xml, > > line 64 > > > > > > What happens if a rebalance command is issued and on

Re: Review Request 57147: Added support for processing custom command script timeout

2017-02-28 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57147/#review167117 --- ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/m

Re: Review Request 57147: Added support for processing custom command script timeout

2017-02-28 Thread Laszlo Puskas
> On Feb. 28, 2017, 6:06 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/metainfo.xml, > > line 64 > > > > > > I don't think Ambari should run a command that takes

Re: Review Request 57147: Added support for processing custom command script timeout

2017-02-28 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57147/#review167110 --- ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/m

Re: Review Request 57147: Added support for processing custom command script timeout

2017-02-28 Thread Alejandro Fernandez
> On Feb. 28, 2017, 6:06 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/metainfo.xml, > > line 64 > > > > > > I don't think Ambari should run a command that takes

Review Request 57147: Added support for processing custom command script timeout

2017-02-28 Thread Laszlo Puskas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57147/ --- Review request for Ambari, Robert Levas, Sandor Magyari, and Sebastian Toader.