Re: Review Request 28405: Distribute Repositories/Install Components - server part

2014-12-03 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28405/ --- (Updated Dec. 3, 2014, 1:55 p.m.) Review request for Ambari, Alejandro

Re: Review Request 28405: Distribute Repositories/Install Components - server part

2014-12-03 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28405/#review63671 --- Ship it! Ship It! - Vitalyi Brodetskyi On Гру. 3, 2014, 1:55

Re: Review Request 28405: Distribute Repositories/Install Components - server part

2014-12-03 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28405/#review63670 --- Ship it! Ship It! - Nate Cole On Dec. 3, 2014, 8:55 a.m.,

Re: Review Request 28405: Distribute Repositories/Install Components - server part

2014-12-02 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28405/ --- (Updated Dec. 2, 2014, 3:46 p.m.) Review request for Ambari, Alejandro

Re: Review Request 28405: Distribute Repositories/Install Components - server part

2014-12-02 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28405/ --- (Updated Dec. 2, 2014, 8:46 p.m.) Review request for Ambari, Alejandro

Re: Review Request 28405: Distribute Repositories/Install Components - server part

2014-11-28 Thread Dmitro Lisnichenko
On Nov. 28, 2014, 2:10 a.m., Nate Cole wrote: ambari-server/src/main/java/org/apache/ambari/server/api/services/HostStackVersionService.java, line 117 https://reviews.apache.org/r/28405/diff/3/?file=777883#file777883line117 What about if just this.clusterName == null? I've used

Re: Review Request 28405: Distribute Repositories/Install Components - server part

2014-11-28 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28405/ --- (Updated Nov. 28, 2014, 6:07 p.m.) Review request for Ambari, Alejandro

Re: Review Request 28405: Distribute Repositories/Install Components - server part

2014-11-27 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28405/ --- (Updated Nov. 27, 2014, 8:17 p.m.) Review request for Ambari, Alejandro

Re: Review Request 28405: Distribute Repositories/Install Components - server part

2014-11-27 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28405/ --- (Updated Nov. 27, 2014, 8:20 p.m.) Review request for Ambari, Alejandro

Re: Review Request 28405: Distribute Repositories/Install Components - server part

2014-11-27 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28405/#review63248 --- wrong patch - Dmitro Lisnichenko On Nov. 27, 2014, 8:20 p.m.,

Re: Review Request 28405: Distribute Repositories/Install Components - server part

2014-11-27 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28405/ --- (Updated Nov. 27, 2014, 8:26 p.m.) Review request for Ambari, Alejandro

Re: Review Request 28405: Distribute Repositories/Install Components - server part

2014-11-27 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28405/#review63253 --- Ship it! Looks good. If this is dependent on another review,

Re: Review Request 28405: Distribute Repositories/Install Components - server part

2014-11-25 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28405/#review62998 --- Ship it! Looks good to me - I'm assuming this will work parallel

Re: Review Request 28405: Distribute Repositories/Install Components - server part

2014-11-25 Thread Dmitro Lisnichenko
On Nov. 25, 2014, 4:53 p.m., Nate Cole wrote: Looks good to me - I'm assuming this will work parallel across all hosts? yes, every host gets his own configuration and all actions are performed in parallel within the same stage - Dmitro

Re: Review Request 28405: Distribute Repositories/Install Components - server part

2014-11-25 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28405/#review63005 ---

Review Request 28405: Distribute Repositories/Install Components - server part

2014-11-24 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28405/ --- Review request for Ambari, Alejandro Fernandez, Nate Cole, and Yurii Shylov.