Review Request 55406: AMBARI-19452 Flume metrics returns same metric values for all channels, sources, sinks

2017-01-10 Thread wang jianfei
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55406/ --- Review request for Ambari, Dmytro Sen, Sumit Mohanty, and Sid Wagle. Summary

Re: Review Request 55174: AMBARI-19360 : Ranger policies not syncing for storm, kafka and knox

2017-01-10 Thread Vishal Suvagia
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55174/ --- (Updated Jan. 11, 2017, 7:18 a.m.) Review request for Ambari, Alejandro

Re: Review Request 55405: AMBARI-19453. Dashboard should display appropriate message if workflow data fetching api fails

2017-01-10 Thread Nitiraj Rathore
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55405/#review161182 --- Ship it! Ship It! - Nitiraj Rathore On Jan. 11, 2017, 6:26

Review Request 55405: AMBARI-19453. Dashboard should display appropriate message if workflow data fetching api fails

2017-01-10 Thread Madhan Reddy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55405/ --- Review request for Ambari, DIPAYAN BHOWMICK, Gaurav Nagar, Nitiraj Rathore, and

Re: Review Request 55382: AMBARI-19386 : ATLAS service check should run against metadata server host.

2017-01-10 Thread Vishal Suvagia
> On Jan. 11, 2017, 12:13 a.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/package/scripts/service_check.py, > > line 37 > > > > > > Why is this iterating over

Review Request 55402: Minor: Ownership is changed before the creation of directory

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

Re: Review Request 55396: Host version incorrectly reported as OUT_OF_SYNC

2017-01-10 Thread Nate Cole
> On Jan. 10, 2017, 8:18 p.m., Jonathan Hurley wrote: > > ambari-server/src/main/java/org/apache/ambari/server/events/listeners/upgrade/HostVersionOutOfSyncListener.java, > > line 170 > > > > > > sp: states Will

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

2017-01-10 Thread ping wang
> On 一月 9, 2017, 7:26 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/OOZIE/4.2.0.2.5/configuration/oozie-site.xml, > > line 22 > > > > > > If the user edited the config in HDP

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

2017-01-10 Thread Weiwei Yang
> On 一月 9, 2017, 7:30 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/package/scripts/service_check.py, > > line 40 > > > > > > Please also include in HDFS

Re: Review Request 55379: Log Feeder should be restarted after configuration change

2017-01-10 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55379/ --- (Updated Jan. 11, 2017, 1:10 a.m.) Review request for Ambari, Oliver Szabo and

Review Request 55399: Changing Capacity-scheduler configs while llap is enabled should warn the user if llap is affected

2017-01-10 Thread Vivek Ratnavel Subramanian
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55399/ --- Review request for Ambari, Jaimin Jetly, Richard Zang, Xi Wang, and Yusaku Sako.

Re: Review Request 55382: AMBARI-19386 : ATLAS service check should run against metadata server host.

2017-01-10 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55382/#review161150 ---

Review Request 55396: Host version incorrectly reported as OUT_OF_SYNC

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

Re: Review Request 55291: AMBARI-19411: Atlas AMS support

2017-01-10 Thread Apoorv Naik
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55291/#review161134 --- Pending metrics.json

Re: Review Request 55291: AMBARI-19411: Atlas AMS support

2017-01-10 Thread Apoorv Naik
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55291/ --- (Updated Jan. 10, 2017, 10:44 p.m.) Review request for Ambari, Aravindan

Re: Review Request 54874: AMBARI-19241 - Ambari python scripts should support hdfs download

2017-01-10 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54874/#review161127 --- Fix it, then Ship it!

Re: Review Request 55372: AMBARI-19437 Remove anonymous bind option for Ranger User sync for LDAP/AD

2017-01-10 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55372/#review161124 --- Ship it! Ship It! - Jonathan Hurley On Jan. 10, 2017, 2:23

Re: Review Request 55391: Role Command Order For HOU Is Different For Some Components

2017-01-10 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55391/#review161120 ---

Review Request 55391: Role Command Order For HOU Is Different For Some Components

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

Re: Review Request 55367: Improve "Install Wizard step 1" after new guidelines

2017-01-10 Thread Jaimin Jetly
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55367/#review161106 --- Ship it! Ship It! - Jaimin Jetly On Jan. 10, 2017, 2:37

Re: Review Request 55367: Improve "Install Wizard step 1" after new guidelines

2017-01-10 Thread Vivek Ratnavel Subramanian
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55367/#review161104 --- Ship it! Ship It! - Vivek Ratnavel Subramanian On Jan. 10,

Re: Review Request 55387: AMBARI-19446: Able to skip writing configuration files when the config type doesn't exist in the command JSON file

2017-01-10 Thread Di Li
> On Jan. 10, 2017, 7:11 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/KNOX/0.5.0.2.2/package/scripts/params_linux.py, > > line 246 > > > > > > -1. If the config doesn't

Re: Review Request 55387: AMBARI-19446: Able to skip writing configuration files when the config type doesn't exist in the command JSON file

2017-01-10 Thread Tim Thorpe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55387/#review161100 --- Ship it! Ship It! - Tim Thorpe On Jan. 10, 2017, 7 p.m., Di

Re: Review Request 55387: AMBARI-19446: Able to skip writing configuration files when the config type doesn't exist in the command JSON file

2017-01-10 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55387/#review161099 ---

Review Request 55387: AMBARI-19446: Able to skip writing configuration files when the config type doesn't exist in the command JSON file

2017-01-10 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55387/ --- Review request for Ambari, Alejandro Fernandez and Tim Thorpe. Bugs:

Re: Review Request 55369: Network interface check returns no value if there is no `ifconfig` command

2017-01-10 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55369/#review161098 --- Fix it, then Ship it!

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

2017-01-10 Thread Alejandro Fernandez
> On Jan. 9, 2017, 7:26 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/OOZIE/4.2.0.2.5/configuration/oozie-site.xml, > > line 22 > > > > > > If the user edited the config in

Re: Review Request 54857: Refactor Ambari service def configurations for Zeppelin

2017-01-10 Thread Renjith Kamath
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54857/ --- (Updated Jan. 10, 2017, 6:16 p.m.) Review request for Ambari, Alejandro

Re: Review Request 55361: LogSearch upgrade support 2.4 -> 2.5 addendum - add log4j properties

2017-01-10 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55361/#review161088 --- Ship it! Ship It! - Robert Nettleton On Jan. 9, 2017, 11:58

Re: Review Request 55384: Handle cancelling of add/delete component by user

2017-01-10 Thread Aleksandr Kovalenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55384/#review161081 --- Ship it! Ship It! - Aleksandr Kovalenko On Янв. 10, 2017,

Re: Review Request 54857: Refactor Ambari service def configurations for Zeppelin

2017-01-10 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54857/#review161078 --- Some unit tests are failing:

Re: Review Request 54857: Refactor Ambari service def configurations for Zeppelin

2017-01-10 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54857/#review161076 --- Ship it! Ship It! - Sumit Mohanty On Jan. 2, 2017, 2:23

Re: Review Request 55364: NodeManager restart fails during HOU if it is on same host as RM

2017-01-10 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55364/#review161074 --- Ship it! Ship It! - Dmitro Lisnichenko On Jan. 10, 2017,

Re: Review Request 55260: Zeppelin stop failed during EU while deleting pid file with customized service user

2017-01-10 Thread Rohit Choudhary
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55260/#review161073 --- Ship it! Ship It! - Rohit Choudhary On Jan. 6, 2017, 9:45

Review Request 55382: AMBARI-19386 : ATLAS service check should run against metadata server host.

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

Re: Review Request 55364: NodeManager restart fails during HOU if it is on same host as RM

2017-01-10 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55364/ --- (Updated Jan. 10, 2017, 10:41 a.m.) Review request for Ambari, Dmitro

Re: Review Request 55378: AMBARI-19439:Warning message when user tries to close a designer tab.

2017-01-10 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55378/#review161072 --- Ship it! Ship It! - Pallav Kulshreshtha On Jan. 10, 2017,

Re: Review Request 55364: NodeManager restart fails during HOU if it is on same host as RM

2017-01-10 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55364/#review161071 --- Ship it! Ship It! - Robert Levas On Jan. 9, 2017, 8:55

Re: Review Request 55342: Modify quick link resource provider to consider filters and return visibility.

2017-01-10 Thread Sandor Magyari
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55342/#review161070 --- Ship it! Ship It! - Sandor Magyari On Jan. 9, 2017, 3:07

Re: Review Request 46268: AMBARI-19044 Install & configure Ranger plugin components independently of Ranger admin components

2017-01-10 Thread Mugdha Varadkar
> On Jan. 9, 2017, 7:24 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/KAFKA/0.9.0/configuration/ranger-kafka-audit.xml, > > line 26 > > > > > > Do all of these properties have

Re: Review Request 55380: Add macros `existsInByKey` and `notExistsInByKey`

2017-01-10 Thread Andrii Tkach
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55380/#review161066 --- Ship it! Ship It! - Andrii Tkach On Jan. 10, 2017, 1:34

Review Request 55380: Add macros `existsInByKey` and `notExistsInByKey`

2017-01-10 Thread Oleg Nechiporenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55380/ --- Review request for Ambari and Andrii Tkach. Bugs: ambari-19445

Re: Review Request 46268: AMBARI-19044 Install & configure Ranger plugin components independently of Ranger admin components

2017-01-10 Thread Mugdha Varadkar
> On Jan. 9, 2017, 7:24 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog250.java, > > line 704 > > > > > > Please add some javadoc. Updated in latest

Re: Review Request 46268: AMBARI-19044 Install & configure Ranger plugin components independently of Ranger admin components

2017-01-10 Thread Mugdha Varadkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46268/ --- (Updated Jan. 10, 2017, 1:31 p.m.) Review request for Ambari, Alejandro

Re: Review Request 55379: Log Feeder should be restarted after configuration change

2017-01-10 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55379/#review161062 --- Ship it! Ship It! - Oliver Szabo On Jan. 10, 2017, 1:09

Review Request 55379: Log Feeder should be restarted after configuration change

2017-01-10 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55379/ --- Review request for Ambari, Oliver Szabo and Robert Nettleton. Bugs:

Re: Review Request 55364: NodeManager restart fails during HOU if it is on same host as RM

2017-01-10 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55364/#review161061 --- Ship it! Ship It! - Nate Cole On Jan. 9, 2017, 8:55 p.m.,

Re: Review Request 55378: AMBARI-19439:Warning message when user tries to close a designer tab.

2017-01-10 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55378/#review161060 --- Ship it! Ship It! - Gaurav Nagar On Jan. 10, 2017, 12:33

Re: Review Request 55375: AMBARI-19440. Need ability to upload and download action asset from HDFS

2017-01-10 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55375/#review161059 --- Ship it! Ship It! - Gaurav Nagar On Jan. 10, 2017, 10:53

Review Request 55378: AMBARI-19439:Warning message when user tries to close a designer tab.

2017-01-10 Thread Padma Priya N
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55378/ --- Review request for Ambari, DIPAYAN BHOWMICK, Gaurav Nagar, Nitiraj Rathore, and

Re: Review Request 55371: AMBARI-19436: Need ability to upload a file in HDFS browser.

2017-01-10 Thread Padma Priya N
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55371/ --- (Updated Jan. 10, 2017, 12:10 p.m.) Review request for Ambari, DIPAYAN

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

2017-01-10 Thread ping wang
> On 一月 9, 2017, 7:26 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/OOZIE/4.2.0.2.5/configuration/oozie-site.xml, > > line 22 > > > > > > If the user edited the config in HDP

Re: Review Request 55376: Ambari should add warning when LogSearch selected without Ambari-Infra

2017-01-10 Thread Oleg Nechiporenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55376/#review161057 --- Ship it! Ship It! - Oleg Nechiporenko On Jan. 10, 2017,

Review Request 55376: Ambari should add warning when LogSearch selected without Ambari-Infra

2017-01-10 Thread Andrii Tkach
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55376/ --- Review request for Ambari and Oleg Nechiporenko. Bugs: AMBARI-19441

Re: Review Request 55174: AMBARI-19288 : Ranger policies not syncing for storm, kafka and knox

2017-01-10 Thread Mugdha Varadkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55174/#review161055 --- Ship it! Ship It! - Mugdha Varadkar On Jan. 9, 2017, 3:12

Re: Review Request 55267: Hive View 2.0 : add REST endpoint for deleting database

2017-01-10 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55267/#review161054 --- Ship it! Ship It! - Gaurav Nagar On Jan. 6, 2017, 2:43

Re: Review Request 54888: Ambari capacity queue manager only accepts values in integers i.e not decimal values like 0.5

2017-01-10 Thread Akhil PB
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54888/ --- (Updated Jan. 10, 2017, 10:07 a.m.) Review request for Ambari, DIPAYAN

Re: Review Request 55267: Hive View 2.0 : add REST endpoint for deleting database

2017-01-10 Thread DIPAYAN BHOWMICK
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55267/#review161053 --- Ship it! Ship It! - DIPAYAN BHOWMICK On Jan. 6, 2017, 2:43

Re: Review Request 55338: Save workflows via save mechanism instead of submission button

2017-01-10 Thread Nitiraj Rathore
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55338/#review161048 --- Ship it! Ship It! - Nitiraj Rathore On Jan. 10, 2017, 9:23

Re: Review Request 55338: Save workflows via save mechanism instead of submission button

2017-01-10 Thread venkat sairam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55338/ --- (Updated Jan. 10, 2017, 9:23 a.m.) Review request for Ambari, DIPAYAN

Re: Review Request 55342: Modify quick link resource provider to consider filters and return visibility.

2017-01-10 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55342/#review161045 --- Ship it!

Review Request 55373: AMBARI-19438 Add enable delta sync property for Ranger

2017-01-10 Thread Mugdha Varadkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55373/ --- Review request for Ambari, Gautam Borad, Jaimin Jetly, Jayush Luniya, and