Re: Review Request 62922: "ambari-server upgrade" failed on db schema [Upgrade]

2017-10-11 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62922/ --- (Updated Oct. 12, 2017, 8:30 a.m.) Review request for Ambari, Jonathan Hurley

Review Request 62922: "ambari-server upgrade" failed on db schema [Upgrade]

2017-10-11 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62922/ --- Review request for Ambari. Bugs: AMBARI-22213

Re: Review Request 62802: AMBARI-22196. Refactor non-stack cluster-env configs to cluster settings.

2017-10-11 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62802/#review187736 --- Ship it! Ship It! - Jayush Luniya On Oct. 11, 2017, 8:26

Re: Review Request 62895: HdpCoreMpack: Quick Links should show NN web UI links

2017-10-11 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62895/#review187702 --- Ship it! Ship It! - Jayush Luniya On Oct. 11, 2017, 4:17

Re: Review Request 62895: HdpCoreMpack: Quick Links should show NN web UI links

2017-10-11 Thread Vitalyi Brodetskyi
> On Жов. 11, 2017, 8:43 після полудня, Jayush Luniya wrote: > > ambari-server/src/main/java/org/apache/ambari/server/state/ServiceInfo.java > > Lines 1075 (patched) > > > > > > Why not set quickLinksConfigurations

Re: Review Request 62895: HdpCoreMpack: Quick Links should show NN web UI links

2017-10-11 Thread Vitalyi Brodetskyi
> On Жов. 11, 2017, 8:43 після полудня, Jayush Luniya wrote: > > ambari-server/src/main/java/org/apache/ambari/server/state/ServiceInfo.java > > Lines 1037 (patched) > > > > > > Why not set themes itself instead of

Re: Review Request 62895: HdpCoreMpack: Quick Links should show NN web UI links

2017-10-11 Thread Jayush Luniya
> On Oct. 11, 2017, 6:45 p.m., Madhuvanthi Radhakrishnan wrote: > > We might want to add it to trunk itself. Thanks The serviceInfo.clone() is something I added for the mpack generator. So the patch won't be applicable there. Lets just fix this in the feature branch. Eventually we will be

Re: Review Request 62895: HdpCoreMpack: Quick Links should show NN web UI links

2017-10-11 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62895/#review187697 ---

Re: Review Request 62802: AMBARI-22196. Refactor non-stack cluster-env configs to cluster settings.

2017-10-11 Thread Swapan Shridhar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62802/ --- (Updated Oct. 11, 2017, 8:26 p.m.) Review request for Ambari, Jayush Luniya

Re: Review Request 62802: AMBARI-22196. Refactor non-stack cluster-env configs to cluster settings.

2017-10-11 Thread Madhuvanthi Radhakrishnan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62802/#review187693 --- Ship it! Ship It! - Madhuvanthi Radhakrishnan On Oct. 11,

Re: Review Request 62802: AMBARI-22196. Refactor non-stack cluster-env configs to cluster settings.

2017-10-11 Thread Swapan Shridhar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62802/ --- (Updated Oct. 11, 2017, 8:24 p.m.) Review request for Ambari, Jayush Luniya

Re: Review Request 62802: AMBARI-22196. Refactor non-stack cluster-env configs to cluster settings.

2017-10-11 Thread Madhuvanthi Radhakrishnan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62802/#review187688 --- Ship it! Ship It! - Madhuvanthi Radhakrishnan On Oct. 11,

Re: Review Request 62904: RU was not started on Oracle due to missing table

2017-10-11 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62904/#review187687 --- Ship it! Ship It! - Jonathan Hurley On Oct. 11, 2017, 3:28

Re: Review Request 62802: AMBARI-22196. Refactor non-stack cluster-env configs to cluster settings.

2017-10-11 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62802/#review187686 --- Ship it! Ship It! - Jayush Luniya On Oct. 11, 2017, 8:13

Re: Review Request 62802: AMBARI-22196. Refactor non-stack cluster-env configs to cluster settings.

2017-10-11 Thread Jayush Luniya
> On Oct. 11, 2017, 4:42 p.m., Madhuvanthi Radhakrishnan wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/RootClusterSettingsResourceProvider.java > > Lines 95 (patched) > > > > > >

Review Request 62904: RU was not started on Oracle due to missing table

2017-10-11 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62904/ --- Review request for Ambari and Jonathan Hurley. Bugs: AMBARI-22211

Re: Review Request 62895: HdpCoreMpack: Quick Links should show NN web UI links

2017-10-11 Thread Madhuvanthi Radhakrishnan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62895/#review187681 --- Ship it! We might want to add it to trunk itself. Thanks -

Re: Review Request 62894: Install Version Fails On IOP Cluster Due To Missing stack_packages Structure

2017-10-11 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62894/#review187676 --- Ship it! Ship It! - Nate Cole On Oct. 11, 2017, 10:45 a.m.,

Re: Review Request 62802: AMBARI-22196. Refactor non-stack cluster-env configs to cluster settings.

2017-10-11 Thread Madhuvanthi Radhakrishnan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62802/#review187664 ---

Re: Review Request 62896: Log Search UI: small fixes for top part of Service Logs page

2017-10-11 Thread Andrii Tkach
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62896/#review187668 --- Ship it! Ship It! - Andrii Tkach On Oct. 11, 2017, 4:30

Review Request 62896: Log Search UI: small fixes for top part of Service Logs page

2017-10-11 Thread Andrii Babiichuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62896/ --- Review request for Ambari and Andrii Tkach. Bugs: AMBARI-22208

Review Request 62895: HdpCoreMpack: Quick Links should show NN web UI links

2017-10-11 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62895/ --- Review request for Ambari, Dmitro Lisnichenko, Jayush Luniya, and Madhuvanthi

Review Request 62894: Install Version Fails On IOP Cluster Due To Missing stack_packages Structure

2017-10-11 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62894/ --- Review request for Ambari, Nate Cole and Robert Levas. Bugs: AMBARI-22205

Re: Review Request 62866: Make hive server create directories related to replication

2017-10-11 Thread Eugene Chekanskiy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62866/ --- (Updated Oct. 11, 2017, 2:44 p.m.) Review request for Ambari, Dmitro

Re: Review Request 62866: Make hive server create directories related to replication

2017-10-11 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62866/#review187658 --- Ship it! Ship It! - Dmitro Lisnichenko On Oct. 11, 2017,

Re: Review Request 62890: AMBARI-22204 Updating Ranger Admin pre-upgrade task

2017-10-11 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62890/#review187657 --- Fix it, then Ship it! Just the one change I mentioned - but in

Re: Review Request 62890: AMBARI-22204 Updating Ranger Admin pre-upgrade task

2017-10-11 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62890/#review187656 --- Ship it! Ship It! - Nate Cole On Oct. 11, 2017, 9:54 a.m.,

Review Request 62890: AMBARI-22204 Updating Ranger Admin pre-upgrade task

2017-10-11 Thread Mugdha Varadkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62890/ --- Review request for Ambari, Alejandro Fernandez, Gautam Borad, Jonathan Hurley,

Re: Review Request 62885: Hive View 2.0 in Ambari 2.5.1 does not use the specified YARN queue when using the "Upload Table" feature.

2017-10-11 Thread DIPAYAN BHOWMICK
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62885/#review187653 --- Ship it! Ship It! - DIPAYAN BHOWMICK On Oct. 11, 2017,

Re: Review Request 62866: Make hive server create directories related to replication

2017-10-11 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62866/#review187651 ---

Re: Review Request 62888: ZKFC start failed due to hadoop-hdfs-zkfc is not supported

2017-10-11 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62888/#review187649 --- Ship it! Ship It! - Vitalyi Brodetskyi On Жов. 11, 2017,

Review Request 62888: ZKFC start failed due to hadoop-hdfs-zkfc is not supported

2017-10-11 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62888/ --- Review request for Ambari and Vitalyi Brodetskyi. Bugs: AMBARI-22203

Re: Review Request 62866: Make hive server create directories related to replication

2017-10-11 Thread Eugene Chekanskiy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62866/ --- (Updated Oct. 11, 2017, 12:09 p.m.) Review request for Ambari, Dmitro

Re: Review Request 62885: Hive View 2.0 in Ambari 2.5.1 does not use the specified YARN queue when using the "Upload Table" feature.

2017-10-11 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62885/#review187644 --- Ship it! Ship It! - Pallav Kulshreshtha On Oct. 11, 2017,

Review Request 62885: Hive View 2.0 in Ambari 2.5.1 does not use the specified YARN queue when using the "Upload Table" feature.

2017-10-11 Thread Nitiraj Rathore
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62885/ --- Review request for Ambari, DIPAYAN BHOWMICK, Pallav Kulshreshtha, and venkat

Re: Review Request 62871: Prevent Patch Upgrades With Known Service Incompatibilities

2017-10-11 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62871/#review187641 --- Fix it, then Ship it!

Re: Review Request 62883: Zeppelin start fails due to permission denied error during kinit

2017-10-11 Thread Prabhjyot Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62883/ --- (Updated Oct. 11, 2017, 2:44 p.m.) Review request for Ambari, DIPAYAN

Re: Review Request 62883: Zeppelin start fails due to permission denied error during kinit

2017-10-11 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62883/#review187640 --- Ship it! Ship It! - Pallav Kulshreshtha On Oct. 11, 2017,

Re: Review Request 62883: Zeppelin start fails due to permission denied error during kinit

2017-10-11 Thread Prabhjyot Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62883/ --- (Updated Oct. 11, 2017, 2:31 p.m.) Review request for Ambari, DIPAYAN

Re: Review Request 62883: Zeppelin start fails due to permission denied error during kinit

2017-10-11 Thread DIPAYAN BHOWMICK
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62883/#review187638 --- Ship it! Ship It! - DIPAYAN BHOWMICK On Oct. 11, 2017, 8:56

Re: Review Request 62883: Zeppelin start fails due to permission denied error during kinit

2017-10-11 Thread venkat sairam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62883/#review187637 --- Ship it! Ship It! - venkat sairam On Oct. 11, 2017, 8:56

Re: Review Request 62883: Zeppelin start fails due to permission denied error during kinit

2017-10-11 Thread Prabhjyot Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62883/ --- (Updated Oct. 11, 2017, 2:26 p.m.) Review request for Ambari, DIPAYAN

Re: Review Request 62802: AMBARI-22196. Refactor non-stack cluster-env configs to cluster settings.

2017-10-11 Thread Swapan Shridhar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62802/ --- (Updated Oct. 11, 2017, 8:13 a.m.) Review request for Ambari, Jayush Luniya

Re: Review Request 62802: AMBARI-22196. Refactor non-stack cluster-env configs to cluster settings.

2017-10-11 Thread Swapan Shridhar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62802/ --- (Updated Oct. 11, 2017, 7:42 a.m.) Review request for Ambari, Jayush Luniya

Re: Review Request 62802: AMBARI-22196. Refactor non-stack cluster-env configs to cluster settings.

2017-10-11 Thread Swapan Shridhar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62802/ --- (Updated Oct. 11, 2017, 7:33 a.m.) Review request for Ambari, Jayush Luniya

Re: Review Request 62802: AMBARI-22196. Refactor non-stack cluster-env configs to cluster settings.

2017-10-11 Thread Swapan Shridhar
> On Oct. 11, 2017, 6:38 a.m., Jayush Luniya wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/spi/Resource.java > > Lines 78 (patched) > > > > > > RootClusterSetting Done. - Swapan

Re: Review Request 62802: AMBARI-22196. Refactor non-stack cluster-env configs to cluster settings.

2017-10-11 Thread Swapan Shridhar
> On Oct. 11, 2017, 6:29 a.m., Jayush Luniya wrote: > > ambari-server/src/main/java/org/apache/ambari/server/api/services/AmbariMetaInfo.java > > Lines 777 (patched) > > > > > > PropertyNotFoundException instead?

Re: Review Request 62802: AMBARI-22196. Refactor non-stack cluster-env configs to cluster settings.

2017-10-11 Thread Swapan Shridhar
> On Oct. 11, 2017, 6:38 a.m., Jayush Luniya wrote: > > ambari-server/src/main/java/org/apache/ambari/server/api/resources/ClusterSettingResourceDefinition.java > > Lines 25 (patched) > > > > > >

Review Request 62883: Zeppelin start fails due to permission denied error during kinit

2017-10-11 Thread Prabhjyot Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62883/ --- Review request for Ambari, DIPAYAN BHOWMICK, Jaimin Jetly, Jayush Luniya, Rohit

Re: Review Request 62802: AMBARI-22196. Refactor non-stack cluster-env configs to cluster settings.

2017-10-11 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62802/#review187631 ---

Re: Review Request 62802: AMBARI-22196. Refactor non-stack cluster-env configs to cluster settings.

2017-10-11 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62802/#review187629 ---