Re: Review Request 63684: Patch Upgrades Broken For Clients Due To Versioned LD Library

2017-11-08 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63684/#review190553 --- Ship it! Ship It! - Dmitro Lisnichenko On Nov. 8, 2017,

Re: Review Request 63573: AMBARI-22366. POST, GET and UPDATE API for cluster settings. (/clusters/{clusterName}/settings).

2017-11-08 Thread Swapan Shridhar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63573/ --- (Updated Nov. 9, 2017, 4:04 a.m.) Review request for Ambari, Jayush Luniya and

Re: Review Request 63573: AMBARI-22366. POST, GET and UPDATE API for cluster settings. (/clusters/{clusterName}/settings).

2017-11-08 Thread Swapan Shridhar
> On Nov. 8, 2017, 2:02 p.m., Attila Doroszlai wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ClusterSettingResourceProvider.java > > Lines 274-282 (patched) > > > > > > This

Re: Review Request 63684: Patch Upgrades Broken For Clients Due To Versioned LD Library

2017-11-08 Thread Dmytro Grinenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63684/#review190532 --- Ship it! Ship It! - Dmytro Grinenko On Nov. 8, 2017, 9:22

Re: Review Request 63653: Add service group dependencies

2017-11-08 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63653/ --- (Updated Лис. 9, 2017, 12:16 до полудня) Review request for Ambari, Jayush

Re: Review Request 63573: AMBARI-22366. POST, GET and UPDATE API for cluster settings. (/clusters/{clusterName}/settings).

2017-11-08 Thread Swapan Shridhar
> On Nov. 8, 2017, 2:02 p.m., Attila Doroszlai wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/ClusterSettingResponse.java > > Lines 126-141 (patched) > > > > > > Can be simplified using

Review Request 63684: Patch Upgrades Broken For Clients Due To Versioned LD Library

2017-11-08 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63684/ --- Review request for Ambari, Dmytro Grinenko, Dmitro Lisnichenko, and Nate Cole.

Re: Review Request 63683: Improve install wizard data handling

2017-11-08 Thread Jason Golieb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63683/ --- (Updated Nov. 8, 2017, 9:19 p.m.) Review request for Ambari, Andrii Tkach,

Re: Review Request 63573: AMBARI-22366. POST, GET and UPDATE API for cluster settings. (/clusters/{clusterName}/settings).

2017-11-08 Thread Swapan Shridhar
> On Nov. 8, 2017, 2:02 p.m., Attila Doroszlai wrote: > > ambari-server/src/main/java/org/apache/ambari/server/api/resources/ClusterSettingResourceDefinition.java > > Lines 37-42 (patched) > > > > > > Why override

Re: Review Request 63671: Remove Auto-Installation of LZO Libraries

2017-11-08 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63671/#review190474 --- Ship it! Ship It! - Sid Wagle On Nov. 8, 2017, 5:08 p.m.,

Re: Review Request 63671: Remove Auto-Installation of LZO Libraries

2017-11-08 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63671/#review190467 --- Ship it! Ship It! - Jonathan Hurley On Nov. 8, 2017, 12:08

Re: Review Request 63640: AMBARI-22378 BI 4.2.5 repoinfo misses the redhat-ppc7 section

2017-11-08 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63640/#review190466 --- Ship it! Ship It! - Jonathan Hurley On Nov. 7, 2017, 12:23

Re: Review Request 63671: Remove Auto-Installation of LZO Libraries

2017-11-08 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63671/#review190464 --- Ship it! Ship It! - Dmitro Lisnichenko On Nov. 8, 2017,

Review Request 63671: Remove Auto-Installation of LZO Libraries

2017-11-08 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63671/ --- Review request for Ambari, Dmitro Lisnichenko, Jonathan Hurley, and Sid Wagle.

Re: Review Request 63640: AMBARI-22378 BI 4.2.5 repoinfo misses the redhat-ppc7 section

2017-11-08 Thread Tim Thorpe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63640/#review190455 --- Ship it! Ship It! - Tim Thorpe On Nov. 7, 2017, 5:23 p.m.,

Review Request 63666: AMBARI-22327 Allow different OS types and versions for Hybrid setup

2017-11-08 Thread Yussuf Shaikh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63666/ --- Review request for Ambari and Nate Cole. Bugs: AMBARI-22327

Re: Review Request 63573: AMBARI-22366. POST, GET and UPDATE API for cluster settings. (/clusters/{clusterName}/settings).

2017-11-08 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63573/#review190435 ---

Re: Review Request 63558: AMBARI-22361: Fix bug in base_alert when matching hostnames.

2017-11-08 Thread stephane san
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63558/ --- (Updated Nov. 8, 2017, 12:30 p.m.) Review request for Ambari, Andrew Onischuk

Re: Review Request 63558: AMBARI-22361: Fix bug in base_alert when matching hostnames.

2017-11-08 Thread stephane san
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63558/ --- (Updated Nov. 8, 2017, 12:27 p.m.) Review request for Ambari, Andrew Onischuk