Re: Review Request 55129: [AMBARI-19337] Ambari has some spelling mistakes in YARN proxyuser properties in many places.

2017-01-17 Thread Sumit Mohanty
> On Jan. 17, 2017, 8:45 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/kerberos.json, > > line 26 > > > > > > This is essentially changing the config name. > >

Review Request 55663: Add support for call context to the stack advisor

2017-01-17 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55663/ --- Review request for Ambari, Jayush Luniya, Swapan Shridhar, and Sid Wagle.

Re: Review Request 55664: AMBARI-19602:Input logic tags are missing in the coordinator xml

2017-01-17 Thread belliraj hb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55664/#review162039 --- Ship it! Ship It! - belliraj hb On Jan. 18, 2017, 7:15

Re: Review Request 55556: ranger_usersync should install with ranger_admin in the same host

2017-01-17 Thread alice alice
> On 一月 17, 2017, 7:09 p.m., Alejandro Fernandez wrote: > > ambari-web/app/models/stack_service_component.js, line 200 > > > > > > The standard is to provide that information in the metainfo file for > > several

Re: Review Request 55556: ranger_usersync should install with ranger_admin in the same host

2017-01-17 Thread alice alice
> On 一月 16, 2017, 7:05 p.m., Alejandro Fernandez wrote: > > ambari-web/app/models/stack_service_component.js, line 200 > > > > > > Can this come from metainfo instead of being hardcoded on the UI? > > > >

Re: Review Request 55627: Hive view 2.0 REST and backend support for enabling and fetching table and column stats

2017-01-17 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55627/#review162036 --- Ship it! Ship It! - Gaurav Nagar On Jan. 17, 2017, 4:18

Re: Review Request 55556: ranger_usersync should install with ranger_admin in the same host

2017-01-17 Thread alice alice
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/6/ --- (Updated 一月 18, 2017, 6:01 a.m.) Review request for Ambari, Alejandro

Re: Review Request 55614: AMBARI-19581. Need to show proper error message in scenarios of failures with asset import

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

Re: Review Request 55627: Hive view 2.0 REST and backend support for enabling and fetching table and column stats

2017-01-17 Thread DIPAYAN BHOWMICK
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55627/#review162030 --- Ship it! Ship It! - DIPAYAN BHOWMICK On Jan. 17, 2017, 4:18

Review Request 55664: AMBARI-19602:Input logic tags are missing in the coordinator xml

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

Re: Review Request 55664: AMBARI-19602:Input logic tags are missing in the coordinator xml

2017-01-17 Thread Nitiraj Rathore
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55664/#review162037 --- Ship it! Ship It! - Nitiraj Rathore On Jan. 18, 2017, 7:15

Re: Review Request 55628: Add/delete host component: visual changes for configuration popup

2017-01-17 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55628/#review161882 --- Ship it! Ship It! - Alexandr Antonenko On Jan. 17, 2017,

Re: Review Request 55630: Hbase, Zookeeper start failed

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

Re: Review Request 55617: AMBARI-19585. Client config tags may not be saved by agent

2017-01-17 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55617/#review161872 --- Ship it! Ship It! - Sebastian Toader On Jan. 17, 2017, 2:28

Re: Review Request 55622: Log Feeder should be able to handle metrics with https

2017-01-17 Thread Miklos Gergely
> On Jan. 17, 2017, 3:57 p.m., Oliver Szabo wrote: > > ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/package/scripts/params.py, > > line 356 > > > > > > im not sure is that will be right

Review Request 55630: Hbase, Zookeeper start failed

2017-01-17 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55630/ --- Review request for Ambari and Dmitro Lisnichenko. Bugs: AMBARI-19593

Re: Review Request 55429: Run ZK operations in logsearch internally and enable ACL security for LogSearch/Solr

2017-01-17 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55429/ --- (Updated Jan. 17, 2017, 5:08 p.m.) Review request for Ambari, Miklos Gergely

Review Request 55627: Hive view 2.0 REST and backend support for enabling and fetching table and column stats

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

Re: Review Request 55629: Ranger Admin HA Wizard: next button disappears

2017-01-17 Thread Andriy Babiichuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55629/#review161883 --- Ship it! Ship It! - Andriy Babiichuk On Січ. 17, 2017, 4:36

Re: Review Request 55597: Add upgrade logic for the heap dump control option added in HDP 2.6 stack.

2017-01-17 Thread Jonathan Hurley
> On Jan. 17, 2017, 9:47 a.m., Robert Levas wrote: > > The implementation accounts only for adding content to a line-delimited > > string value. Would there be a need to insert a value into a string with a > > different delimiater? Maybe a comma (,)? No, I don't think so. Consider this: ``

Review Request 55628: Add/delete host component: visual changes for configuration popup

2017-01-17 Thread Andriy Babiichuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55628/ --- Review request for Ambari and Alexandr Antonenko. Bugs: AMBARI-19590

Re: Review Request 55596: AMBARI-19576 - Downgrade request fails as existing Upgrade request is considered 'in progress'

2017-01-17 Thread Xi Wang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55596/#review161894 --- Ship it! Ship It! - Xi Wang On Jan. 16, 2017, 6:17 p.m.,

Re: Review Request 55597: Add upgrade logic for the heap dump control option added in HDP 2.6 stack.

2017-01-17 Thread Jonathan Hurley
> On Jan. 17, 2017, 9:47 a.m., Robert Levas wrote: > > The implementation accounts only for adding content to a line-delimited > > string value. Would there be a need to insert a value into a string with a > > different delimiater? Maybe a comma (,)? > > Jonathan Hurley wrote: > No, I

Re: Review Request 55597: Add upgrade logic for the heap dump control option added in HDP 2.6 stack.

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

Re: Review Request 55630: Hbase, Zookeeper start failed

2017-01-17 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55630/#review161892 --- Ship it! Ship It! - Jayush Luniya On Jan. 17, 2017, 4:38

Re: Review Request 55597: Add upgrade logic for the heap dump control option added in HDP 2.6 stack.

2017-01-17 Thread Jonathan Hurley
> On Jan. 17, 2017, 9:47 a.m., Robert Levas wrote: > > ambari-server/src/main/java/org/apache/ambari/server/serveraction/upgrades/ConfigureAction.java, > > line 479 > > > > > > `contains` seems to be a little

Re: Review Request 55516: Workflow Manager View: Start record set to 1 even when there are no records

2017-01-17 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55516/#review161901 --- Ship it! Ship It! - Di Li On Jan. 17, 2017, 5:53 p.m.,

Re: Review Request 55556: ranger_usersync should install with ranger_admin in the same host

2017-01-17 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/6/#review161910 --- ambari-web/app/models/stack_service_component.js (line 200)

Re: Review Request 55516: Workflow Manager View: Start record set to 1 even when there are no records

2017-01-17 Thread DIPAYAN BHOWMICK
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55516/#review161916 ---

Re: Review Request 55597: Add upgrade logic for the heap dump control option added in HDP 2.6 stack.

2017-01-17 Thread Robert Levas
> On Jan. 17, 2017, 9:47 a.m., Robert Levas wrote: > > ambari-server/src/main/java/org/apache/ambari/server/serveraction/upgrades/ConfigureAction.java, > > line 479 > > > > > > `contains` seems to be a little

Review Request 55634: ZKFC version shows incorrect in DB tables upon downgrade

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

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

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

Re: Review Request 55429: Run ZK operations in logsearch internally and enable ACL security for LogSearch/Solr

2017-01-17 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55429/ --- (Updated Jan. 17, 2017, 8:06 p.m.) Review request for Ambari, Miklos Gergely

Re: Review Request 55129: [AMBARI-19337] Ambari has some spelling mistakes in YARN proxyuser properties in many places.

2017-01-17 Thread Sumit Mohanty
> On Jan. 17, 2017, 8:08 p.m., Nate Cole wrote: > > Ping. I just committed it. Jay can you close it. - Sumit --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55129/#review161930

Re: Review Request 55634: ZKFC version shows incorrect in DB tables upon downgrade

2017-01-17 Thread Jonathan Hurley
> On Jan. 17, 2017, 3:07 p.m., Jonathan Hurley wrote: > > ambari-server/src/main/java/org/apache/ambari/server/state/UpgradeHelper.java, > > lines 814-817 > > > > > > Do you think it's better to simply do: > >

Re: Review Request 55429: Run ZK operations in logsearch internally and enable ACL security for LogSearch/Solr

2017-01-17 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55429/ --- (Updated Jan. 17, 2017, 8:16 p.m.) Review request for Ambari, Miklos Gergely

Re: Review Request 55597: Add upgrade logic for the heap dump control option added in HDP 2.6 stack.

2017-01-17 Thread Robert Levas
> On Jan. 17, 2017, 9:47 a.m., Robert Levas wrote: > > ambari-server/src/main/java/org/apache/ambari/server/serveraction/upgrades/ConfigureAction.java, > > line 479 > > > > > > `contains` seems to be a little

Re: Review Request 55634: ZKFC version shows incorrect in DB tables upon downgrade

2017-01-17 Thread Nate Cole
> On Jan. 17, 2017, 3:07 p.m., Jonathan Hurley wrote: > > ambari-server/src/main/java/org/apache/ambari/server/state/UpgradeHelper.java, > > lines 791-793 > > > > > > OK, this is a really odd question and I'm not

Re: Review Request 55129: [AMBARI-19337] Ambari has some spelling mistakes in YARN proxyuser properties in many places.

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

Re: Review Request 55538: AMBARI-19545: Ambari-agent - In HIVE and OOZIE stack scripts, copy JCEKS file to desired location

2017-01-17 Thread Nahappan Somasundaram
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55538/ --- (Updated Jan. 17, 2017, 12:48 p.m.) Review request for Ambari, Andrew

Review Request 55629: Ranger Admin HA Wizard: next button disappears

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

Re: Review Request 55622: Log Feeder should be able to handle metrics with https

2017-01-17 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55622/#review161884 --- Ship it! Ship It! - Oliver Szabo On Jan. 17, 2017, 2:44

Review Request 55642: AMBARI-19598. (1). Set the calculated value for config 'num_llap_nodes', and (2). Remove 'Tech Preview' string for HSI from HDP 2.6 onwards.

2017-01-17 Thread Swapan Shridhar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55642/ --- Review request for Ambari, Alejandro Fernandez, Sumit Mohanty, and Siddharth

Re: Review Request 55597: Add upgrade logic for the heap dump control option added in HDP 2.6 stack.

2017-01-17 Thread Jonathan Hurley
> On Jan. 17, 2017, 9:47 a.m., Robert Levas wrote: > > ambari-server/src/main/java/org/apache/ambari/server/serveraction/upgrades/ConfigureAction.java, > > line 479 > > > > > > `contains` seems to be a little

Re: Review Request 55634: ZKFC version shows incorrect in DB tables upon downgrade

2017-01-17 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55634/ --- (Updated Jan. 17, 2017, 2:59 p.m.) Review request for Ambari, Dmitro

Re: Review Request 55597: Add upgrade logic for the heap dump control option added in HDP 2.6 stack.

2017-01-17 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55597/#review161923 --- Ship it! Ship It! - Nate Cole On Jan. 16, 2017, 9:54 p.m.,

Re: Review Request 55538: AMBARI-19545: Ambari-agent - In HIVE and OOZIE stack scripts, copy JCEKS file to desired location

2017-01-17 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55538/#review161924 --- Fix it, then Ship it!

Re: Review Request 55634: ZKFC version shows incorrect in DB tables upon downgrade

2017-01-17 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55634/#review161927 --- Ship it! Ship It! - Alejandro Fernandez On Jan. 17, 2017,

Re: Review Request 55634: ZKFC version shows incorrect in DB tables upon downgrade

2017-01-17 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55634/ --- (Updated Jan. 17, 2017, 3:29 p.m.) Review request for Ambari, Dmitro

Re: Review Request 55590: Add common log rotation settings to HDFS, Zookeeper configs - addendum

2017-01-17 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55590/#review161943 --- Ship it! Ship It! - Sumit Mohanty On Jan. 16, 2017, 8:19

Re: Review Request 55597: Add upgrade logic for the heap dump control option added in HDP 2.6 stack.

2017-01-17 Thread Jonathan Hurley
> On Jan. 17, 2017, 2:13 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/java/org/apache/ambari/server/serveraction/upgrades/ConfigureAction.java, > > line 498 > > > > > > What do you think about

Re: Review Request 55597: Add upgrade logic for the heap dump control option added in HDP 2.6 stack.

2017-01-17 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55597/#review161920 --- Ship it! Ship It! - Alejandro Fernandez On Jan. 17, 2017,

Re: Review Request 55129: [AMBARI-19337] Ambari has some spelling mistakes in YARN proxyuser properties in many places.

2017-01-17 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55129/#review161930 --- Ping. - Nate Cole On Jan. 3, 2017, 3:41 a.m., Jay SenSharma

Re: Review Request 55634: ZKFC version shows incorrect in DB tables upon downgrade

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

Review Request 55651: AMS log4j rotation properties changes should be visible on ambari-upgrade

2017-01-17 Thread Madhuvanthi Radhakrishnan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55651/ --- Review request for Ambari and Sumit Mohanty. Bugs: AMBARI-19600

Re: Review Request 55647: Fix issues around modifying yarn min container size

2017-01-17 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55647/#review161999 --- Ship it! Ship It! - Alejandro Fernandez On Jan. 17, 2017,

Re: Review Request 55556: ranger_usersync should install with ranger_admin in the same host

2017-01-17 Thread Jayush Luniya
> On Jan. 17, 2017, 7:09 p.m., Alejandro Fernandez wrote: > > ambari-web/app/models/stack_service_component.js, line 200 > > > > > > The standard is to provide that information in the metainfo file for > > several

Re: Review Request 55642: AMBARI-19598. (1). Set the calculated value for config 'num_llap_nodes', and (2). Remove 'Tech Preview' string for HSI from HDP 2.6 onwards.

2017-01-17 Thread Swapan Shridhar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55642/ --- (Updated Jan. 17, 2017, 9:04 p.m.) Review request for Ambari, Alejandro

Re: Review Request 53686: Stage and Request status should be persisted in the database

2017-01-17 Thread Jaimin Jetly
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53686/ --- (Updated Jan. 17, 2017, 9:43 p.m.) Review request for Ambari, Jonathan Hurley,

Re: Review Request 53686: Stage and Request status should be persisted in the database

2017-01-17 Thread Jaimin Jetly
> On Dec. 15, 2016, 3:40 p.m., Nate Cole wrote: > > ambari-server/src/main/java/org/apache/ambari/server/actionmanager/ActionScheduler.java, > > line 186 > > > > > > What I meant was this isn't a typically-named

Re: Review Request 55429: Run ZK operations in logsearch internally and enable ACL security for LogSearch/Solr

2017-01-17 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55429/#review161972 --- Fix it, then Ship it! Ship It!

Re: Review Request 55642: AMBARI-19598. (1). Set the calculated value for config 'num_llap_nodes', and (2). Remove 'Tech Preview' string for HSI from HDP 2.6 onwards.

2017-01-17 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55642/#review161946 --- Ship it! Ship It! - Sumit Mohanty On Jan. 17, 2017, 8:58

Review Request 55643: Blueprint installation should accept quick link profile

2017-01-17 Thread Balázs Bence Sári
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55643/ --- Review request for Ambari, Attila Doroszlai, Laszlo Puskas, Oliver Szabo,

Re: Review Request 55642: AMBARI-19598. (1). Set the calculated value for config 'num_llap_nodes', and (2). Remove 'Tech Preview' string for HSI from HDP 2.6 onwards.

2017-01-17 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55642/#review161948 --- Ship it! Ship It! - Alejandro Fernandez On Jan. 17, 2017,

Re: Review Request 55643: Blueprint installation should accept quick link profile

2017-01-17 Thread Balázs Bence Sári
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55643/ --- (Updated Jan. 17, 2017, 9:30 p.m.) Review request for Ambari, Attila

Re: Review Request 55516: Workflow Manager View: Start record set to 1 even when there are no records

2017-01-17 Thread Sangeeta Ravindran
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55516/ --- (Updated Jan. 17, 2017, 9:45 p.m.) Review request for Ambari, Di Li, DIPAYAN

Re: Review Request 55597: Add upgrade logic for the heap dump control option added in HDP 2.6 stack.

2017-01-17 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55597/#review161868 --- Fix it, then Ship it! The implementation accounts only for

Review Request 55622: Log Feeder should be able to handle metrics with https

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

Re: Review Request 55538: AMBARI-19545: Ambari-agent - In HIVE and OOZIE stack scripts, copy JCEKS file to desired location

2017-01-17 Thread Nahappan Somasundaram
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55538/ --- (Updated Jan. 17, 2017, 5:03 p.m.) Review request for Ambari, Andrew Onischuk,

Re: Review Request 55651: AMS log4j rotation properties changes should be visible on ambari-upgrade

2017-01-17 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55651/#review162011 --- Ship it! Ship It! - Sumit Mohanty On Jan. 17, 2017, 11:07

Re: Review Request 55556: ranger_usersync should install with ranger_admin in the same host

2017-01-17 Thread alice zhangxiaolu
> On 一月 17, 2017, 7:09 p.m., Alejandro Fernandez wrote: > > ambari-web/app/models/stack_service_component.js, line 200 > > > > > > The standard is to provide that information in the metainfo file for > > several

Re: Review Request 55556: ranger_usersync should install with ranger_admin in the same host

2017-01-17 Thread Alejandro Fernandez
> On Jan. 17, 2017, 7:09 p.m., Alejandro Fernandez wrote: > > ambari-web/app/models/stack_service_component.js, line 200 > > > > > > The standard is to provide that information in the metainfo file for > > several

Re: Review Request 55538: AMBARI-19545: Ambari-agent - In HIVE and OOZIE stack scripts, copy JCEKS file to desired location

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

Re: Review Request 55538: AMBARI-19545: Ambari-agent - In HIVE and OOZIE stack scripts, copy JCEKS file to desired location

2017-01-17 Thread Nahappan Somasundaram
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55538/ --- (Updated Jan. 17, 2017, 2:55 p.m.) Review request for Ambari, Andrew Onischuk,

Review Request 55609: Add Livy session recovery configurations in Ambari

2017-01-17 Thread Saisai Shao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55609/ --- Review request for Ambari and Sumit Mohanty. Bugs: AMBARI-19577

Re: Review Request 55556: ranger_usersync should install with ranger_admin in the same host

2017-01-17 Thread alice zhangxiaolu
> On 一月 16, 2017, 7:05 p.m., Alejandro Fernandez wrote: > > ambari-web/app/models/stack_service_component.js, line 200 > > > > > > Can this come from metainfo instead of being hardcoded on the UI? > > > >

Re: Review Request 55555: Ambari Yarn Queue Manager allows two same queue name

2017-01-17 Thread Sreenath Somarajapuram
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5/#review161841 --- Ship it! Ship It! - Sreenath Somarajapuram On Jan. 16,

Re: Review Request 55566: AMBARI-19562: Save is not present for coordinator and bundle.

2017-01-17 Thread Nitiraj Rathore
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55566/#review161831 --- Ship it! Ship It! - Nitiraj Rathore On Jan. 16, 2017, 9:44

Re: Review Request 55573: Hive View 2.0: Export Query Result

2017-01-17 Thread Nitiraj Rathore
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55573/#review161837 --- Ship it! Ship It! - Nitiraj Rathore On Jan. 16, 2017, 12:39

Re: Review Request 55566: AMBARI-19562: Save is not present for coordinator and bundle.

2017-01-17 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55566/#review161833 --- Ship it! Ship It! - Gaurav Nagar On Jan. 16, 2017, 9:44

Re: Review Request 55567: AMBARI-19563:Dashboard-Show YARN RM URL in actions and flow graph and organize tabs

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

Re: Review Request 55555: Ambari Yarn Queue Manager allows two same queue name

2017-01-17 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5/#review161842 --- Ship it! Ship It! - Gaurav Nagar On Jan. 16, 2017, 6:35

Re: Review Request 55614: AMBARI-19581. Need to show proper error message in scenarios of failures with asset import

2017-01-17 Thread Nitiraj Rathore
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55614/#review161856 --- Ship it! Ship It! - Nitiraj Rathore On Jan. 17, 2017, 12:02

Review Request 55617: AMBARI-19585. Client config tags may not be saved by agent

2017-01-17 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55617/ --- Review request for Ambari, Andrew Onischuk, Sandor Magyari, and Sebastian

Re: Review Request 55614: AMBARI-19581. Need to show proper error message in scenarios of failures with asset import

2017-01-17 Thread belliraj hb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55614/#review161855 --- Ship it! Ship It! - belliraj hb On Jan. 17, 2017, 12:02

Review Request 55615: Hive View 2.0: Show query logs

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

Review Request 55610: AMBARI-19580: Organize tabs in job details page

2017-01-17 Thread Padma Priya N
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55610/ --- Review request for Ambari, belliraj hb, Gaurav Nagar, and Pallav Kulshreshtha.

Re: Review Request 55573: Hive View 2.0: Export Query Result

2017-01-17 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55573/#review161851 --- Ship it! Ship It! - Gaurav Nagar On Jan. 16, 2017, 12:39

Re: Review Request 55610: AMBARI-19580: Organize tabs in job details page

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

Review Request 55614: AMBARI-19581. Need to show proper error message in scenarios of failures with asset import

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

Re: Review Request 55556: ranger_usersync should install with ranger_admin in the same host

2017-01-17 Thread alice zhangxiaolu
> On 一月 16, 2017, 7:05 p.m., Alejandro Fernandez wrote: > > ambari-web/app/models/stack_service_component.js, line 200 > > > > > > Can this come from metainfo instead of being hardcoded on the UI? > > > >

Re: Review Request 55618: Ranger Admin HA Wizard should display configs changes explicitly

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

Review Request 55618: Ranger Admin HA Wizard should display configs changes explicitly

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

Re: Review Request 53213: Exporting blueprint from kerberos enabled cluster, exports hardcoded values cluster name and realm in principal_name property

2017-01-17 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53213/#review161866 --- Ship it! Ship It! - Robert Levas On Jan. 11, 2017, 7:33

Re: Review Request 55647: Fix issues around modifying yarn min container size

2017-01-17 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55647/ --- (Updated Jan. 18, 2017, 5:03 a.m.) Review request for Ambari, Siddharth Seth

Re: Review Request 55129: [AMBARI-19337] Ambari has some spelling mistakes in YARN proxyuser properties in many places.

2017-01-17 Thread Jay SenSharma
> On Jan. 17, 2017, 8:08 p.m., Nate Cole wrote: > > Ping. > > Sumit Mohanty wrote: > I just committed it. Jay can you close it. Sumit, i see 'Alejandro' has some queries related to the property change to existing clusters. Once that discussion is over then we can close this review. -