Re: Review Request 53576: AMBARI-18569 Execute topology tasks in parallel by hosts

2016-11-10 Thread Attila Doroszlai
> On Nov. 10, 2016, 5:18 p.m., Jonathan Hurley wrote: > > ambari-server/src/main/java/org/apache/ambari/server/configuration/Configuration.java, > > line 1 > > > > > > I think if we want to change this, it should be

Re: Review Request 53576: AMBARI-18569 Execute topology tasks in parallel by hosts

2016-11-10 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53576/#review155601 ---

Re: Review Request 53576: AMBARI-18569 Execute topology tasks in parallel by hosts

2016-11-10 Thread Attila Doroszlai
> On Nov. 10, 2016, 11:04 a.m., Sebastian Toader wrote: > > ambari-server/src/main/java/org/apache/ambari/server/configuration/Configuration.java, > > line 1 > > > > > > Add * to be consistent with the licensing

Re: Review Request 53576: AMBARI-18569 Execute topology tasks in parallel by hosts

2016-11-10 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53576/#review155560 ---

Re: Review Request 53576: AMBARI-18569 Execute topology tasks in parallel by hosts

2016-11-09 Thread Attila Doroszlai
> On Nov. 9, 2016, 3:55 p.m., Jonathan Hurley wrote: > > ambari-server/src/main/java/org/apache/ambari/server/topology/TopologyManager.java, > > line 740 > > > > > > Do either the ClusterTopology or

Re: Review Request 53576: AMBARI-18569 Execute topology tasks in parallel by hosts

2016-11-09 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53576/ --- (Updated Nov. 9, 2016, 4:07 p.m.) Review request for Ambari, Jonathan Hurley,

Re: Review Request 53576: AMBARI-18569 Execute topology tasks in parallel by hosts

2016-11-09 Thread Attila Doroszlai
> On Nov. 9, 2016, 2:42 p.m., Laszlo Puskas wrote: > > ambari-server/src/main/java/org/apache/ambari/server/topology/TopologyManager.java, > > line 736 > > > > > > Check the configuration for parallel execution

Re: Review Request 53576: AMBARI-18569 Execute topology tasks in parallel by hosts

2016-11-09 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53576/#review155436 --- Fix it, then Ship it!

Re: Review Request 53576: AMBARI-18569 Execute topology tasks in parallel by hosts

2016-11-09 Thread Laszlo Puskas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53576/#review155432 ---

Re: Review Request 53576: AMBARI-18569 Execute topology tasks in parallel by hosts

2016-11-09 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53576/#review155422 --- Ship it!

Re: Review Request 53576: AMBARI-18569 Execute topology tasks in parallel by hosts

2016-11-09 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53576/ --- (Updated Nov. 9, 2016, 11:27 a.m.) Review request for Ambari, Jonathan Hurley,

Re: Review Request 53576: AMBARI-18569 Execute topology tasks in parallel by hosts

2016-11-08 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53576/#review155319 --- Ship it! Ship It! - Alejandro Fernandez On Nov. 8, 2016,

Re: Review Request 53576: AMBARI-18569 Execute topology tasks in parallel by hosts

2016-11-08 Thread Sandor Magyari
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53576/#review155316 --- Ship it! Ship It! - Sandor Magyari On Nov. 8, 2016, 5:14

Review Request 53576: AMBARI-18569 Execute topology tasks in parallel by hosts

2016-11-08 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53576/ --- Review request for Ambari, Jonathan Hurley, Sandor Magyari, and Sebastian