Re: Review Request 54895: AMBARI-19256:Asset support Rest API

2016-12-22 Thread Nitiraj Rathore
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54895/#review160042 --- Ship it! Ship It! - Nitiraj Rathore On Dec. 21, 2016, 8:57

Re: Review Request 54977: Zeppelin Notebook Component is not able start after reboot

2016-12-22 Thread Nitiraj Rathore
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54977/#review160041 --- Ship it! Ship It! - Nitiraj Rathore On Dec. 22, 2016, 12:55

Re: Review Request 55009: HDFS Service check fails if previous active NN is down

2016-12-22 Thread Weiwei Yang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55009/ --- (Updated 十二月 23, 2016, 7:13 a.m.) Review request for Ambari and Di Li. Bugs:

Re: Review Request 55007: AMBARI-19288 : Do not install hive-hook from Ambari if falcon client is installed

2016-12-22 Thread Mugdha Varadkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55007/#review160036 --- Ship it! Ship It! - Mugdha Varadkar On Dec. 23, 2016, 5:23

Re: Review Request 54817: AMBARI-19227 :: Provide support to show Ranger solr-configs in Ambari.

2016-12-22 Thread Vishal Suvagia
> On Dec. 21, 2016, 6:20 p.m., Oliver Szabo wrote: > > Ship It! > > 1 note: is it okay to add to the > > properties? Maybe im wrong, but if the ranger-solr-configuration wont exist > > after ambari upgrade, the solr related config references can fail Thank you Oliver, As the

Review Request 55007: AMBARI-19288 : Do not install hive-hook from Ambari if falcon client is installed

2016-12-22 Thread Vishal Suvagia
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55007/ --- Review request for Ambari, Alejandro Fernandez, Gautam Borad, Jaimin Jetly,

Re: Review Request 54958: Single API to download all client configs

2016-12-22 Thread Jaimin Jetly
> On Dec. 22, 2016, 3:59 p.m., Nate Cole wrote: > > ambari-server/src/main/java/org/apache/ambari/server/api/services/HostComponentService.java, > > line 299 > > > > > > Would rather use new File(parent, child)

Re: Review Request 54958: Single API to download all client configs

2016-12-22 Thread Jaimin Jetly
> On Dec. 22, 2016, 5:32 a.m., Sid Wagle wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ClientConfigResourceProvider.java, > > line 250 > > > > > > Suggestion friendlier naming

Re: Review Request 54958: Single API to download all client configs

2016-12-22 Thread Jaimin Jetly
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54958/ --- (Updated Dec. 23, 2016, 12:57 a.m.) Review request for Ambari, Jonathan

Re: Review Request 54958: Single API to download all client configs

2016-12-22 Thread Jaimin Jetly
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54958/ --- (Updated Dec. 23, 2016, 12:55 a.m.) Review request for Ambari, Jonathan

Review Request 55000: Perf: Deploy 3000 Agent cluster and find perf bugs. Part 4

2016-12-22 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55000/ --- Review request for Ambari, Alejandro Fernandez, Sumit Mohanty, and Sid Wagle.

Re: Review Request 54942: AMBARI-19272. Ignored mount points logged for each mount

2016-12-22 Thread Dmytro Grinenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54942/#review160018 --- Ship it! Ship It! - Dmytro Grinenko On Dec. 21, 2016, 4:52

Re: Review Request 54955: Add common log4j settings to advanced config - hbase, zookeeper, ams

2016-12-22 Thread Madhuvanthi Radhakrishnan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54955/ --- (Updated Dec. 22, 2016, 10:24 p.m.) Review request for Ambari and Sumit

Re: Review Request 54994: Enable alert scripts from stack extensions

2016-12-22 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54994/#review160012 --- Ship it! Ship It! - Di Li On Dec. 22, 2016, 8:59 p.m.,

Re: Review Request 54994: Enable alert scripts from stack extensions

2016-12-22 Thread Tim Thorpe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54994/#review160011 --- Ship it! Ship It! - Tim Thorpe On Dec. 22, 2016, 8:59 p.m.,

Review Request 54994: Enable alert scripts from stack extensions

2016-12-22 Thread Diego Santesteban
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54994/ --- Review request for Ambari, Di Li and Tim Thorpe. Bugs: AMBARI-19283

Re: Review Request 54936: Zookeeper and RM connection is not secure

2016-12-22 Thread Attila Magyar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54936/ --- (Updated Dec. 22, 2016, 8:53 p.m.) Review request for Ambari, Attila

Re: Review Request 54989: HOU pre-checks indicate HA as a requirement

2016-12-22 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54989/ --- (Updated Dec. 22, 2016, 3:03 p.m.) Review request for Ambari, Dmitro

Review Request 54989: HOU pre-checks indicate HA as a requirement

2016-12-22 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54989/ --- Review request for Ambari, Dmitro Lisnichenko, Nate Cole, and Robert Levas.

Re: Review Request 54966: Fix typo in druid service postgresql

2016-12-22 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54966/#review159976 --- Ship it! Ship It! - Sebastian Toader On Dec. 22, 2016, 5:31

Re: Review Request 54966: Fix typo in druid service postgresql

2016-12-22 Thread Attila Magyar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54966/#review159975 --- Ship it! Ship It! - Attila Magyar On Dec. 22, 2016, 4:31

Re: Review Request 54966: Fix typo in druid service postgresql

2016-12-22 Thread Slim Bouguerra
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54966/ --- (Updated Dec. 22, 2016, 4:31 p.m.) Review request for Ambari, Attila Magyar,

Re: Review Request 54982: Zeppelin Notebook start fails after reboot

2016-12-22 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54982/#review159974 --- Ship it! Ship It! - Vitalyi Brodetskyi On Гру. 22, 2016,

Review Request 54982: Zeppelin Notebook start fails after reboot

2016-12-22 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54982/ --- Review request for Ambari and Vitalyi Brodetskyi. Bugs: AMBARI-19284

Re: Review Request 54970: AMBARI-19250 Change default Spark configuration property in Oozie configuration settings to valid value

2016-12-22 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54970/#review159973 ---

Re: Review Request 54970: AMBARI-19250 Change default Spark configuration property in Oozie configuration settings to valid value

2016-12-22 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54970/#review159972 --- Hello Ping, Please also add Alejandro Fernandez (@afernandez)for

Re: Review Request 54958: Single API to download all client configs

2016-12-22 Thread Nate Cole
> On Dec. 22, 2016, 12:32 a.m., Sid Wagle wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ClientConfigResourceProvider.java, > > line 240 > > > > > > Shouldn't this just log and

Re: Review Request 54958: Single API to download all client configs

2016-12-22 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54958/#review159970 ---

Re: Review Request 54979: Typo in the PostgreSQL database name for druid service

2016-12-22 Thread Slim Bouguerra
> On Dec. 22, 2016, 2:55 p.m., Slim Bouguerra wrote: > > also i have sent the patch to Apache JIRA > > https://issues.apache.org/jira/browse/AMBARI-19278 but it is not applied. > > Attila Magyar wrote: > I think it was not applied because it was created from the ambari-server > directory

Re: Review Request 54979: Typo in the PostgreSQL database name for druid service

2016-12-22 Thread Attila Magyar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54979/ --- (Updated Dec. 22, 2016, 3:16 p.m.) Review request for Ambari, Attila

Re: Review Request 54981: HA HDFS propreties may be written in different case, which prevents operations

2016-12-22 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54981/#review159965 --- Ship it! Ship It! - Dmitro Lisnichenko On Dec. 22, 2016,

Re: Review Request 54979: Typo in the PostgreSQL database name for druid service

2016-12-22 Thread Attila Magyar
> On Dec. 22, 2016, 2:55 p.m., Slim Bouguerra wrote: > > also i have sent the patch to Apache JIRA > > https://issues.apache.org/jira/browse/AMBARI-19278 but it is not applied. > > Attila Magyar wrote: > I think it was not applied because it was created from the ambari-server > directory

Review Request 54981: HA HDFS propreties may be written in different case, which prevents operations

2016-12-22 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54981/ --- Review request for Ambari and Dmitro Lisnichenko. Bugs: AMBARI-19282

Re: Review Request 54979: Typo in the PostgreSQL database name for druid service

2016-12-22 Thread Slim Bouguerra
> On Dec. 22, 2016, 2:55 p.m., Slim Bouguerra wrote: > > also i have sent the patch to Apache JIRA > > https://issues.apache.org/jira/browse/AMBARI-19278 but it is not applied. > > Attila Magyar wrote: > I think it was not applied because it was created from the ambari-server > directory

Re: Review Request 54979: Typo in the PostgreSQL database name for druid service

2016-12-22 Thread Attila Magyar
> On Dec. 22, 2016, 2:55 p.m., Slim Bouguerra wrote: > > also i have sent the patch to Apache JIRA > > https://issues.apache.org/jira/browse/AMBARI-19278 but it is not applied. I think it was not applied because it was created from the ambari-server directory instead of the project root. -

Re: Review Request 54979: Typo in the PostgreSQL database name for druid service

2016-12-22 Thread Slim Bouguerra
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54979/#review159961 --- also i have sent the patch to Apache JIRA

Re: Review Request 54979: Typo in the PostgreSQL database name for druid service

2016-12-22 Thread Slim Bouguerra
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54979/#review159960 --- i have sennt a PR alreadsy https://reviews.apache.org/r/54966/

Review Request 54979: Typo in the PostgreSQL database name for druid service

2016-12-22 Thread Attila Magyar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54979/ --- Review request for Ambari, Attila Doroszlai, Balázs Bence Sári, Slim Bouguerra,

Review Request 54977: Zeppelin Notebook Component is not able start after reboot

2016-12-22 Thread Renjith Kamath
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54977/ --- Review request for Ambari, Alejandro Fernandez, DIPAYAN BHOWMICK, Jayush

Re: Review Request 54976: Display properties for all affected services before adding or deleting ZooKeeper Server

2016-12-22 Thread Aleksandr Kovalenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54976/#review159955 --- Ship it! Ship It! - Aleksandr Kovalenko On Дек. 22, 2016,

Review Request 54976: Display properties for all affected services before adding or deleting ZooKeeper Server

2016-12-22 Thread Andriy Babiichuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54976/ --- Review request for Ambari and Aleksandr Kovalenko. Bugs: AMBARI-19281

Re: Review Request 54973: Cover global controllers with unit tests

2016-12-22 Thread Aleksandr Kovalenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54973/#review159946 --- Ship it! Ship It! - Aleksandr Kovalenko On Дек. 22, 2016,

Review Request 54973: Cover global controllers with unit tests

2016-12-22 Thread Andrii Tkach
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54973/ --- Review request for Ambari and Aleksandr Kovalenko. Bugs: AMBARI-19280

Re: Review Request 54935: ulimit override missing from upgrade catalogs

2016-12-22 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54935/ --- (Updated Дек. 22, 2016, 10:49 д.п.) Review request for Ambari, Sumit Mohanty,

Re: Review Request 54892: AMBARI-19254 : Prompting for underlying workflow job properties in a coordinator and coordinator job properties in a bundle during job submission

2016-12-22 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54892/#review159942 --- Ship it! Ship It! - Gaurav Nagar On Dec. 20, 2016, 12:32

Re: Review Request 54971: AHW Configurations page looks not like any other wizard page

2016-12-22 Thread Aleksandr Kovalenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54971/#review159939 --- Ship it! Ship It! - Aleksandr Kovalenko On Дек. 22, 2016,

Review Request 54971: AHW Configurations page looks not like any other wizard page

2016-12-22 Thread Oleg Nechiporenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54971/ --- Review request for Ambari and Aleksandr Kovalenko. Bugs: ambari-19279

Re: Review Request 54936: Zookeeper and RM connection is not secure

2016-12-22 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54936/#review159938 --- Ship it! Ship It! - Sebastian Toader On Dec. 22, 2016,

Review Request 54970: AMBARI-19250 Change default Spark configuration property in Oozie configuration settings to valid value

2016-12-22 Thread ping wang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54970/ --- Review request for Ambari and Di Li. Bugs: AMBARI-19250

Re: Review Request 54936: Zookeeper and RM connection is not secure

2016-12-22 Thread Attila Magyar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54936/ --- (Updated Dec. 22, 2016, 9:14 a.m.) Review request for Ambari, Attila