Re: Review Request 47882: HBase backups fail if there is no /user/hbase directory in HDFS

2016-05-26 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47882/#review134982 --- Ship it! Ship It! - Vitalyi Brodetskyi On Травень 26, 2016,

Re: Review Request 47856: AMBARI-16887 : [AMS / Grafana] Metrics are staying flat for 1 minutes, causing rate calculations to be 0

2016-05-26 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47856/#review134988 --- Fix it, then Ship it!

Re: Review Request 47879: AMBARI-16807: After enabling HTTPS for HDFS, Data Node JVM Metrics on HDFS Heatmaps show NA

2016-05-26 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47879/#review134989 --- Ship it! Ship It! - Dmytro Sen On Май 26, 2016, 10:27 д.п.,

Re: Review Request 47018: "ambari-server upgrade" shouldn't automatically add stack configs

2016-05-26 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47018/ --- (Updated May 26, 2016, 7:15 p.m.) Review request for Ambari, Alejandro

Re: Review Request 47871: Support Storm 1.0 in EU/RU to HDP 2.5

2016-05-26 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47871/#review135001 --- Ship it!

Re: Review Request 47867: RU: install version should be blocked while upgrade in progress

2016-05-26 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47867/#review134997 --- Ship it! Ship It!

Re: Review Request 47879: AMBARI-16807: After enabling HTTPS for HDFS, Data Node JVM Metrics on HDFS Heatmaps show NA

2016-05-26 Thread Qin Liu
> On May 26, 2016, 3:11 p.m., Dmytro Sen wrote: > > Ship It! Thank you so much Dmytro! - Qin --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47879/#review134989

Re: Review Request 47836: User/Group with no Cluster Role assigned but having View Permissions of "VIEW.USER" are shown as not editable in the List View

2016-05-26 Thread Keta Patel
> On May 25, 2016, 10:50 p.m., Richard Zang wrote: > > Ship It! Thank you Richard! - Keta --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47836/#review134858

Review Request 47858: Cache service advisors when stack advisor is loaded

2016-05-26 Thread Lav Jain
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47858/ --- Review request for Ambari, Alexander Denissov, bhuvnesh chaudhary, Jayush

Re: Review Request 47856: AMBARI-16887 : [AMS / Grafana] Metrics are staying flat for 1 minutes, causing rate calculations to be 0

2016-05-26 Thread Aravindan Vijayan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47856/ --- (Updated May 26, 2016, 7:16 p.m.) Review request for Ambari, Dmytro Sen, Sumit

Re: Review Request 47880: AMBARI-16890 Updating Ambari configs changes for latest Ranger configs

2016-05-26 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47880/#review135014 ---

Re: Review Request 47884: Ambari install failure due to zeppelin service install bug

2016-05-26 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47884/#review135012 ---

Re: Review Request 47856: AMBARI-16887 : [AMS / Grafana] Metrics are staying flat for 1 minutes, causing rate calculations to be 0

2016-05-26 Thread Aravindan Vijayan
> On May 26, 2016, 3:10 p.m., Dmytro Sen wrote: > > ambari-metrics/ambari-metrics-timelineservice/src/main/java/org/apache/hadoop/yarn/server/applicationhistoryservice/metrics/timeline/aggregators/TimelineMetricClusterAggregatorSecond.java, > > line 279 > >

Re: Review Request 47871: Support Storm 1.0 in EU/RU to HDP 2.5

2016-05-26 Thread Alejandro Fernandez
> On May 26, 2016, 4:42 p.m., Jonathan Hurley wrote: > > ambari-server/src/main/resources/stacks/HDP/2.3/upgrades/upgrade-2.5.xml, > > lines 860-863 > > > > > > The Jira says: > > ``` > > Delete Storm data

Re: Review Request 47858: Cache service advisors when stack advisor is loaded

2016-05-26 Thread Tim Thorpe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47858/#review135049 --- Ship it! Ship It! - Tim Thorpe On May 26, 2016, 6:50 p.m.,

Re: Review Request 47774: AMBARI-16843. Auto logout zeppelin server in case of session inactivity.

2016-05-26 Thread Prabhjyot Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47774/ --- (Updated May 26, 2016, 7:18 a.m.) Review request for Ambari, Alejandro

Re: Review Request 47722: AMBARI-16813 Ranger Usersync config to support Group Based Search for LDAP Sync Source

2016-05-26 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47722/#review134934 --- Ship it! Ship It! - Jayush Luniya On May 25, 2016, 10:38

Re: Review Request 47738: [Hive View] Non-visual explain gives a bunch of strangely clickable links

2016-05-26 Thread DIPAYAN BHOWMICK
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47738/#review134945 --- Ship it! Ship It! - DIPAYAN BHOWMICK On May 26, 2016, 7:44

Re: Review Request 47738: [Hive View] Non-visual explain gives a bunch of strangely clickable links

2016-05-26 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47738/ --- (Updated May 26, 2016, 7:44 a.m.) Review request for Ambari, DIPAYAN BHOWMICK,

Re: Review Request 47738: [Hive View] Non-visual explain gives a bunch of strangely clickable links

2016-05-26 Thread Pallav Kulshreshtha
> On May 24, 2016, 1:08 p.m., Nitiraj Rathore wrote: > > contrib/views/hive/src/main/resources/ui/hive-web/app/controllers/index/history-query/explain.js, > > line 135 > > > > > > Instead of Array.map() you should

Re: Review Request 47579: Spark2 service definition for Ambari

2016-05-26 Thread Jayush Luniya
> On May 25, 2016, 8:29 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/SPARK2/2.0.0/package/scripts/setup_spark.py, > > line 25 > > > > > > Use explicit import Sorry I missed

Re: Review Request 47913: VDF builder script and XSD should be updated for package-version changes

2016-05-26 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47913/#review135052 --- Ship it! Ship It! - Alejandro Fernandez On May 26, 2016,

Re: Review Request 47871: Support Storm 1.0 in EU/RU to HDP 2.5

2016-05-26 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47871/#review135055 --- Ship it! Ship It! - Jonathan Hurley On May 26, 2016, 4:03

Review Request 47913: VDF builder script and XSD should be updated for package-version changes

2016-05-26 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47913/ --- Review request for Ambari, Alejandro Fernandez and Jonathan Hurley. Bugs:

Review Request 47915: Oozie and Hive Server start fail during EU with missing DB class exception

2016-05-26 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47915/ --- Review request for Ambari, Andrew Onischuk, Dmitro Lisnichenko, and Sumit

Re: Review Request 47913: VDF builder script and XSD should be updated for package-version changes

2016-05-26 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47913/#review135051 --- Ship it! contrib/version-builder/example.sh (lines 39 - 40)

Re: Review Request 47871: Support Storm 1.0 in EU/RU to HDP 2.5

2016-05-26 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47871/ --- (Updated May 26, 2016, 8:03 p.m.) Review request for Ambari, Andrew Onischuk,

Review Request 47920: Enable HS2 Interactive > Select Hosts does not have Cancel and close

2016-05-26 Thread Jaimin Jetly
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47920/ --- Review request for Ambari, Zhe (Joe) Wang, Richard Zang, Xi Wang, and Yusaku

Review Request 47923: clean up import * for SPARK service scripts in common-services

2016-05-26 Thread Juanjo Marron
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47923/ --- Review request for Ambari, Alejandro Fernandez, Jayush Luniya, and Matt. Bugs:

Review Request 47922: clean up import * for KNOX service scripts in common-services

2016-05-26 Thread Juanjo Marron
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47922/ --- Review request for Ambari, Alejandro Fernandez, Jayush Luniya, and Matt. Bugs:

Review Request 47933: Add conditional constraints for Kerberos identities to control when they are created

2016-05-26 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47933/ --- Review request for Ambari, Jonathan Hurley, Nate Cole, Oliver Szabo, and Sandor

Review Request 47926: Use postgres database instead of template1 during service check in HAWQ

2016-05-26 Thread bhuvnesh chaudhary
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47926/ --- Review request for Ambari, Alexander Denissov, Lav Jain, and Matt. Bugs:

Review Request 47941: [AMBARI-16920] Spark2 thrift server can not started due to miss of spark-thrift-fairscheduler.xml

2016-05-26 Thread Jeff Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47941/ --- Review request for Ambari, Jayush Luniya and Sumit Mohanty. Bugs: AMBARI-16920

Re: Review Request 47880: AMBARI-16890 Updating Ambari configs changes for latest Ranger configs

2016-05-26 Thread Mugdha Varadkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47880/ --- (Updated May 26, 2016, 11:50 a.m.) Review request for Ambari, Alejandro

Review Request 47884: Ambari install failure due to zeppelin service install bug

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

Re: Review Request 47783: Cluster operator and cluster admin not allowed to install ambari agent

2016-05-26 Thread Robert Levas
> On May 25, 2016, 4:21 p.m., Jonathan Hurley wrote: > > ambari-server/src/main/resources/Ambari-DDL-SQLServer-CREATE.sql, line 1296 > > > > > > The description says that both the cluster operator and the admin

Review Request 47882: HBase backups fail if there is no /user/hbase directory in HDFS

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

Review Request 47885: AMBARI-16894: Default Ranger repos for some services are not getting created

2016-05-26 Thread Gautam Borad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47885/ --- Review request for Ambari, Alejandro Fernandez, Jaimin Jetly, Jayush Luniya,

Re: Review Request 47920: Enable HS2 Interactive > Select Hosts does not have Cancel and close

2016-05-26 Thread Yusaku Sako
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47920/#review135117 --- Ship it! Ship It! - Yusaku Sako On May 26, 2016, 10:36

Review Request 47934: clean up import * for SLIDER service scripts in common-services

2016-05-26 Thread Juanjo Marron
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47934/ --- Review request for Ambari, Alejandro Fernandez, Jayush Luniya, and Matt. Bugs:

Re: Review Request 46727: [AMBARI-15612] Add Livy to HDP 2.5 as slave component of Spark

2016-05-26 Thread Jeff Zhang
> On May 18, 2016, 11:53 p.m., Jayush Luniya wrote: > > @Jeff > > Can you verify that blueprint deployment of the cluster without Livy Server > > works? > > > > Guide on Blueprints > > https://cwiki.apache.org/confluence/display/AMBARI/Blueprints > > Jayush Luniya wrote: > @Jeff whats

Re: Review Request 47939: AMBARI-16919. Handle the scenario when 'capacity-scheduler' is passed-in as dictionary to Stack Advisor and services['configurations'][capacity-scheduler][properties][capacit

2016-05-26 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47939/#review135142 --- Ship it! Ship It! - Sumit Mohanty On May 27, 2016, 2:03

Re: Review Request 47914: AMBARI-16910. Hive Server Interactive. Change the timeout to 120 secs for LLAP alert command.

2016-05-26 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47914/#review135141 --- Ship it! Ship It! - Sumit Mohanty On May 27, 2016, 12:49

Re: Review Request 47926: Use postgres database instead of template1 during service check in HAWQ

2016-05-26 Thread Lav Jain
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47926/#review135116 --- Ship it! Ship It! - Lav Jain On May 26, 2016, 11:24 p.m.,

Re: Review Request 47914: AMBARI-16910. Hive Server Interactive. Change the timeout to 120 secs for LLAP alert command.

2016-05-26 Thread Swapan Shridhar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47914/ --- (Updated May 27, 2016, 12:49 a.m.) Review request for Ambari, Alejandro

Review Request 47914: AMBARI-16910. Hive Server Interactive. Change the timeout to 120 secs for LLAP alert command.

2016-05-26 Thread Swapan Shridhar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47914/ --- Review request for Ambari, Alejandro Fernandez and Sumit Mohanty. Bugs:

Re: Review Request 47783: Cluster operator and cluster admin not allowed to install ambari agent

2016-05-26 Thread Robert Levas
> On May 25, 2016, 4:21 p.m., Jonathan Hurley wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/RequestResourceProvider.java, > > lines 190-194 > > > > > > This is a little odd -

Review Request 47880: AMBARI-16890 Updating Ambari configs changes for latest Ranger configs

2016-05-26 Thread Mugdha Varadkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47880/ --- Review request for Ambari, Alejandro Fernandez, Gautam Borad, Jaimin Jetly,

Review Request 47879: AMBARI-16807: After enabling HTTPS for HDFS, Data Node JVM Metrics on HDFS Heatmaps show NA

2016-05-26 Thread Qin Liu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47879/ --- Review request for Ambari, Dmytro Sen and Sid Wagle. Bugs: AMBARI-16807

Re: Review Request 47867: RU: install version should be blocked while upgrade in progress

2016-05-26 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47867/ --- (Updated May 26, 2016, 6:47 a.m.) Review request for Ambari, Alejandro

Re: Review Request 47867: RU: install version should be blocked while upgrade in progress

2016-05-26 Thread Nate Cole
> On May 25, 2016, 10:02 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ClusterStackVersionResourceProvider.java, > > line 287 > > > > > > Nitpicking

Review Request 47886: Log Search Server start failed with KeeperException

2016-05-26 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47886/ --- Review request for Ambari, Miklos Gergely, Robert Nettleton, and Sumit Mohanty.

Re: Review Request 47886: Log Search Server start failed with KeeperException

2016-05-26 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47886/#review134967 --- Ship it! Ship It! - Robert Nettleton On May 26, 2016, 1:11

Re: Review Request 47886: Log Search Server start failed with KeeperException

2016-05-26 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47886/#review134970 --- Ship it! Ship It! - Miklos Gergely On May 26, 2016, 1:11

Re: Review Request 47882: HBase backups fail if there is no /user/hbase directory in HDFS

2016-05-26 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47882/#review134971 --- Ship it! Ship It! - Sumit Mohanty On May 26, 2016, 12:19

Re: Review Request 47886: Log Search Server start failed with KeeperException

2016-05-26 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47886/ --- (Updated May 26, 2016, 1:47 p.m.) Review request for Ambari, Miklos Gergely,

Re: Review Request 47871: Support Storm 1.0 in EU/RU to HDP 2.5

2016-05-26 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47871/#review134974 --- Ship it!

Re: Review Request 47785: Ambari install of Atlas should use external HBase and Logsearch SOLR

2016-05-26 Thread Nate Cole
> On May 25, 2016, 7:30 p.m., Srimanth Gunturi wrote: > > ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/package/scripts/params.py, > > line 150 > > > > > > Just wanted to make sure if this always

Re: Review Request 47785: Ambari install of Atlas should use external HBase and Logsearch SOLR

2016-05-26 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47785/#review134975 --- Ship it! Ship It! - Nate Cole On May 25, 2016, 6:06 p.m.,