Re: Review Request 55230: HOU Fails To Restart NameNode in non-HA Cluster

2017-01-05 Thread Dmytro Grinenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55230/#review160687 --- ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/p

Review Request 55257: AMBARI-19398 : Namenode not resolved while submitting workflow in certain environments

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

Re: Review Request 55206: AMBARI-19385: Resolve variables in workflow if workflow path in coordinator contains variables.

2017-01-05 Thread DIPAYAN BHOWMICK
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55206/#review160684 --- Ship it! Ship It! - DIPAYAN BHOWMICK On Jan. 5, 2017, 12:14

Re: Review Request 55209: ATS service check failed on navigating to Hive View

2017-01-05 Thread DIPAYAN BHOWMICK
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55209/#review160681 --- Ship it! Ship It! - DIPAYAN BHOWMICK On Jan. 5, 2017, 1:30 p

Review Request 55250: AMBARI-19384. Wrong placeholder in REST metrics error message

2017-01-05 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55250/ --- Review request for Ambari and Jonathan Hurley. Bugs: AMBARI-19384 https://i

Re: Review Request 55206: AMBARI-19385: Resolve variables in workflow if workflow path in coordinator contains variables.

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

Re: Review Request 55206: AMBARI-19385: Resolve variables in workflow if workflow path in coordinator contains variables.

2017-01-05 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55206/#review160678 --- Ship it! Ship It! - Gaurav Nagar On Jan. 5, 2017, 12:14 p.m.

Review Request 55249: Infra Solr Upgrade support 2.4 -> 2.5

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

Re: Review Request 55245: ambari-web action to download all client configs of a service

2017-01-05 Thread Yusaku Sako
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55245/#review160663 --- Ship it! Ship It! - Yusaku Sako On Jan. 6, 2017, 1:21 a.m.,

Review Request 55245: ambari-web action to download all client configs of a service

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

Review Request 55244: Relax checkstyle rules since causing unit test failures

2017-01-05 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55244/ --- Review request for Ambari. Bugs: AMBARI-19395 https://issues.apache.org/jir

Re: Review Request 55233: Improve and Fix "Hosts page" after new guidelines

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

Re: Review Request 55232: Grafana datasource changes to accommodate new visualizations for TopN

2017-01-05 Thread Vivek Ratnavel Subramanian
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55232/ --- (Updated Jan. 5, 2017, 11:44 p.m.) Review request for Ambari, Jaimin Jetly, Ric

Re: Review Request 55233: Improve and Fix "Hosts page" after new guidelines

2017-01-05 Thread Jaimin Jetly
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55233/#review160643 --- Ship it! Ship It! - Jaimin Jetly On Jan. 5, 2017, 10:18 p.m.

Re: Review Request 55234: AMBARI-19393 - Cluster User role is able to view auto start page

2017-01-05 Thread Xi Wang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55234/#review160640 --- Ship it! Ship It! - Xi Wang On Jan. 5, 2017, 2:42 p.m., Rich

Review Request 55234: AMBARI-19393 - Cluster User role is able to view auto start page

2017-01-05 Thread Richard Zang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55234/ --- Review request for Ambari and Xi Wang. Bugs: AMBARI-19393 https://issues.ap

Review Request 55233: Improve and Fix "Hosts page" after new guidelines

2017-01-05 Thread Xi Wang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55233/ --- Review request for Ambari, Jaimin Jetly, Richard Zang, Vivek Ratnavel Subramania

Review Request 55232: Grafana datasource changes to accommodate new visualizations for TopN

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

Review Request 55230: HOU Fails To Restart NameNode in non-HA Cluster

2017-01-05 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55230/ --- Review request for Ambari, Dmytro Grinenko and Nate Cole. Bugs: AMBARI-19344

Re: Review Request 55230: HOU Fails To Restart NameNode in non-HA Cluster

2017-01-05 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55230/#review160635 --- ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/p

Review Request 55231: Authentication negotiation HTTP response should be sent when Kerberos authentication is enabled

2017-01-05 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55231/ --- Review request for Ambari, Attila Magyar, Jonathan Hurley, Laszlo Puskas, and Na

Review Request 55229: AMBARI-19392. Status command executor may use obsolete settings

2017-01-05 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55229/ --- Review request for Ambari, Dmytro Sen, Myroslav Papirkovskyy, Sumit Mohanty, Seb

Re: Review Request 55207: Heartbeat version transitions must update Component alongside Cluster

2017-01-05 Thread Nate Cole
> On Jan. 5, 2017, 1:59 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/java/org/apache/ambari/server/orm/entities/HostComponentStateEntity.java, > > line 73 > > > > > > What should the behavior be if the

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

2017-01-05 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55009/#review160618 --- ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/p

Re: Review Request 55207: Heartbeat version transitions must update Component alongside Cluster

2017-01-05 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55207/#review160613 --- ambari-server/src/main/java/org/apache/ambari/server/orm/entities

Re: Review Request 55207: Heartbeat version transitions must update Component alongside Cluster

2017-01-05 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55207/ --- (Updated Jan. 5, 2017, 1:37 p.m.) Review request for Ambari, Alejandro Fernande

Re: Review Request 55213: PERF: disabling security fails

2017-01-05 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55213/#review160608 --- Ship it! Ship It! - Sid Wagle On Jan. 5, 2017, 3:32 p.m., Dm

Re: Review Request 55207: Heartbeat version transitions must update Component alongside Cluster

2017-01-05 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55207/ --- (Updated Jan. 5, 2017, 12:51 p.m.) Review request for Ambari, Alejandro Fernand

Re: Review Request 55182: Add common log rotation settings to ranger, ranger-kms and falcon

2017-01-05 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55182/#review160606 --- Ship it! Ship It! - Sumit Mohanty On Jan. 5, 2017, 4:33 p.m.

Re: Review Request 55182: Add common log rotation settings to ranger, ranger-kms and falcon

2017-01-05 Thread Madhuvanthi Radhakrishnan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55182/ --- (Updated Jan. 5, 2017, 4:33 p.m.) Review request for Ambari and Sumit Mohanty.

Re: Review Request 55207: Heartbeat version transitions must update Component alongside Cluster

2017-01-05 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55207/#review160602 --- ambari-server/src/main/java/org/apache/ambari/server/state/Servic

Re: Review Request 55207: Heartbeat version transitions must update Component alongside Cluster

2017-01-05 Thread Nate Cole
> On Jan. 5, 2017, 11:10 a.m., Jonathan Hurley wrote: > > ambari-server/src/main/resources/Ambari-DDL-Oracle-CREATE.sql, line 185 > > > > > > Typo; extra NOT. Should be: > > > > repo_state VARCHAR2(255) DEFA

Re: Review Request 55207: Heartbeat version transitions must update Component alongside Cluster

2017-01-05 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55207/#review160598 --- Fix it, then Ship it! ambari-server/src/main/java/org/apache/a

Re: Review Request 55193: Executing ambari-server unit tests with JDK 1.8 results in unit test failures

2017-01-05 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55193/#review160594 --- Ship it! Ship It! - Nate Cole On Jan. 4, 2017, 8:50 p.m., Ro

Re: Review Request 55107: LogSearch upgrade support 2.4 -> 2.5

2017-01-05 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55107/#review160593 --- Ship it! Ship It! - Robert Nettleton On Jan. 5, 2017, 3:02 a

Review Request 55213: PERF: disabling security fails

2017-01-05 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55213/ --- Review request for Ambari, Myroslav Papirkovskyy, Sid Wagle, and Vitalyi Brodets

Re: Review Request 55212: When deleting services UI shows a lot of configs to be changing but none changes

2017-01-05 Thread Oleg Nechiporenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55212/#review160588 --- Ship it! Ship It! - Oleg Nechiporenko On Янв. 5, 2017, 2:49

Re: Review Request 55193: Executing ambari-server unit tests with JDK 1.8 results in unit test failures

2017-01-05 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55193/#review160589 --- Ship it! Ship It! - Jonathan Hurley On Jan. 4, 2017, 8:50 p.

Review Request 55212: When deleting services UI shows a lot of configs to be changing but none changes

2017-01-05 Thread Andriy Babiichuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55212/ --- Review request for Ambari and Oleg Nechiporenko. Bugs: AMBARI-19387 https:/

Re: Review Request 55210: Config Groups: Save button not enabled while trying to overwrite configs

2017-01-05 Thread Andrii Tkach
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55210/#review160584 --- Ship it! Ship It! - Andrii Tkach On Jan. 5, 2017, 1:46 p.m.,

Review Request 55210: Config Groups: Save button not enabled while trying to overwrite configs

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

Re: Review Request 55209: ATS service check failed on navigating to Hive View

2017-01-05 Thread Rohit Choudhary
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55209/#review160583 --- Ship it! Ship It! - Rohit Choudhary On Jan. 5, 2017, 1:30 p.

Review Request 55209: ATS service check failed on navigating to Hive View

2017-01-05 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55209/ --- Review request for Ambari, DIPAYAN BHOWMICK, Nitiraj Rathore, Pallav Kulshreshth

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

2017-01-05 Thread Vishal Suvagia
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55174/ --- (Updated Jan. 5, 2017, 1:24 p.m.) Review request for Ambari, Alejandro Fernande

Review Request 55207: Heartbeat version transitions must update Component alongside Cluster

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

Review Request 55206: AMBARI-19385: Resolve variables in workflow if workflow path in coordinator contains variables.

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

Re: Review Request 55205: Configs changes after adding/deleting host component: value shouldn't be changed if it differs in host names order only

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

Review Request 55205: Configs changes after adding/deleting host component: value shouldn't be changed if it differs in host names order only

2017-01-05 Thread Andriy Babiichuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55205/ --- Review request for Ambari and Aleksandr Kovalenko. Bugs: AMBARI-19383 https

Re: Review Request 55203: AMBARI-19380. Workflow Draft Save-Not working

2017-01-05 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55203/#review160578 --- Ship it! Ship It! - Gaurav Nagar On Jan. 5, 2017, 10:02 a.m.

Re: Review Request 55195: solr_cloud_util.py should update solr data in ZK if it has changed

2017-01-05 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55195/#review160576 --- ambari-common/src/main/python/resource_management/libraries/funct

Re: Review Request 55203: AMBARI-19380. Workflow Draft Save-Not working

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

Re: Review Request 55203: AMBARI-19380. Workflow Draft Save-Not working

2017-01-05 Thread Nitiraj Rathore
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55203/#review160575 --- Ship it! Ship It! - Nitiraj Rathore On Jan. 5, 2017, 10:02 a

Review Request 55203: AMBARI-19380. Workflow Draft Save-Not working

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

Re: Review Request 55195: solr_cloud_util.py should update solr data in ZK if it has changed

2017-01-05 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55195/ --- (Updated Jan. 5, 2017, 9:45 a.m.) Review request for Ambari, Oliver Szabo and R

Re: Review Request 55170: UI elements misalignment in dashboard screen of WFD

2017-01-05 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55170/#review160573 --- Ship it! Ship It! - Gaurav Nagar On Jan. 4, 2017, 12:32 p.m.

Re: Review Request 55170: UI elements misalignment in dashboard screen of WFD

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

Re: Review Request 55196: Not able to save FS action node after populating job xml value

2017-01-05 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55196/#review160570 --- Ship it! Ship It! - Gaurav Nagar On Jan. 5, 2017, 8:34 a.m.,

Re: Review Request 55196: Not able to save FS action node after populating job xml value

2017-01-05 Thread Nitiraj Rathore
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55196/#review160569 --- Ship it! Ship It! - Nitiraj Rathore On Jan. 5, 2017, 8:34 a.

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

2017-01-05 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55087/#review160567 --- Ship it! Ship It! - Gaurav Nagar On Dec. 29, 2016, 5:09 p.m.

Re: Review Request 55196: Not able to save FS action node after populating job xml value

2017-01-05 Thread venkat sairam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55196/ --- (Updated Jan. 5, 2017, 8:34 a.m.) Review request for Ambari, DIPAYAN BHOWMICK,

Review Request 55196: Not able to save FS action node after populating job xml value

2017-01-05 Thread venkat sairam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55196/ --- Review request for Ambari, DIPAYAN BHOWMICK, Gaurav Nagar, Nitiraj Rathore, Pall