Review Request 55833: Supporting zookeeper security only from HDP 2.6

2017-01-23 Thread Attila Magyar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55833/ --- Review request for Ambari, Laszlo Puskas, Robert Levas, and Sebastian Toader.

Re: Review Request 55833: Supporting zookeeper security only from HDP 2.6

2017-01-23 Thread Attila Magyar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55833/ --- (Updated Jan. 23, 2017, 10:03 a.m.) Review request for Ambari, Laszlo Puskas,

Re: Review Request 55833: Supporting zookeeper security only from HDP 2.6

2017-01-23 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55833/#review162610 --- Ship it! Ship It! - Robert Levas On Jan. 23, 2017, 5:03

Re: Review Request 55833: Supporting zookeeper security only from HDP 2.6

2017-01-23 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55833/#review162608 ---

Re: Review Request 55831: Hive view loosing one row every 100 row in result set.

2017-01-23 Thread Rohit Choudhary
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55831/#review162609 --- Ship it! Ship It! - Rohit Choudhary On Jan. 23, 2017, 7:15

Review Request 55834: AMBARI-19669:Not able to add the settings for action nodes in firefox browser

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

Review Request 55836: Hive View 2.0: Editor should be stretchable by dragging

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

Review Request 55838: Alert Popup doesn't execute callback if it's closed without click on "Primary"

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

Re: Review Request 55818: Unknown attributes should not be allowed in quick link filter definitions

2017-01-23 Thread Sandor Magyari
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55818/#review162617 --- Ship it! Ship It! - Sandor Magyari On Jan. 23, 2017, 10:27

Review Request 55839: Hive View 2.0: Code hints for query editor are not working

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

Re: Review Request 55834: AMBARI-19669:Not able to add the settings for action nodes in firefox browser

2017-01-23 Thread belliraj hb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55834/#review162619 --- Ship it! Ship It! - belliraj hb On Jan. 23, 2017, 9:53

Re: Review Request 55718: AMBARI-19624:Missing fields for workflow SLA

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

Re: Review Request 55821: Log Search: support credential store api (part 1)

2017-01-23 Thread Oliver Szabo
> On Jan. 23, 2017, 7:26 a.m., Sebastian Toader wrote: > > ambari-logsearch/ambari-logsearch-logfeeder/src/main/java/org/apache/ambari/logfeeder/util/SSLUtil.java, > > line 129 > > > > > > Isn't this an error that

Re: Review Request 55838: Alert Popup doesn't execute callback if it's closed without click on "Primary"

2017-01-23 Thread Andrii Tkach
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55838/#review162614 --- Ship it! Ship It! - Andrii Tkach On Jan. 23, 2017, 11:33

Re: Review Request 55840: AMBARI-19674. Asset Manager import and validation issues

2017-01-23 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55840/#review162622 --- Ship it! Ship It! - Gaurav Nagar On Jan. 23, 2017, 11:59

Re: Review Request 55831: Hive view loosing one row every 100 row in result set.

2017-01-23 Thread DIPAYAN BHOWMICK
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55831/#review162611 --- Ship it! Ship It! - DIPAYAN BHOWMICK On Jan. 23, 2017, 7:15

Review Request 55837: Hive View 2.0: Incorrect behaviour on select of database

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

Re: Review Request 55818: Unknown attributes should not be allowed in quick link filter definitions

2017-01-23 Thread Balázs Bence Sári
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55818/ --- (Updated Jan. 23, 2017, 10:27 a.m.) Review request for Ambari, Attila

Re: Review Request 55765: llow user to add custom property for log4j

2017-01-23 Thread Andrii Tkach
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55765/ --- (Updated Jan. 23, 2017, 10:36 a.m.) Review request for Ambari, Madhuvanthi

Re: Review Request 55823: Log Search: Add basic auth support to Swagger UI

2017-01-23 Thread Sandor Magyari
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55823/#review162620 --- Ship it! Ship It! - Sandor Magyari On Jan. 23, 2017, 10:48

Re: Review Request 55822: Log Search: Snapshot tour fix & configurable CORS filter

2017-01-23 Thread Sandor Magyari
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55822/#review162621 --- Ship it! Ship It! - Sandor Magyari On Jan. 23, 2017, 10:49

Re: Review Request 55821: Log Search: support credential store api (part 1)

2017-01-23 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55821/#review162626 --- Ship it! Ship It! - Sebastian Toader On Jan. 22, 2017, 7:57

Re: Review Request 55834: AMBARI-19669:Not able to add the settings for action nodes in firefox browser

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

Review Request 55842: Regenerate Keytabs action does not set cluster-env/security_enabled to true

2017-01-23 Thread Eugene Chekanskiy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55842/ --- Review request for Ambari, Dmitro Lisnichenko, Robert Levas, and Vitalyi

Re: Review Request 55833: Supporting zookeeper security only from HDP 2.6

2017-01-23 Thread Attila Magyar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55833/ --- (Updated Jan. 23, 2017, 1:25 p.m.) Review request for Ambari, Laszlo Puskas,

Review Request 55844: ZKFC start failed

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

Re: Review Request 55837: Hive View 2.0: Incorrect behaviour on select of database

2017-01-23 Thread Abhishek Kumar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55837/ --- (Updated Jan. 23, 2017, 11:37 a.m.) Review request for Ambari, DIPAYAN

Re: Review Request 55836: Hive View 2.0: Editor should be stretchable by dragging

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

Review Request 55840: AMBARI-19674. Asset Manager import and validation issues

2017-01-23 Thread Madhan Reddy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55840/ --- Review request for Ambari, belliraj hb, Gaurav Nagar, and Rohit Choudhary.

Re: Review Request 55840: AMBARI-19674. Asset Manager import and validation issues

2017-01-23 Thread belliraj hb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55840/#review162618 --- Ship it! Ship It! - belliraj hb On Jan. 23, 2017, 11:59

Re: Review Request 55845: Storm service check failed after HOU to HDP-2.5.3 due to CNF for StormAtlasHook

2017-01-23 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55845/#review162642 --- Ship it! Ship It! - Nate Cole On Jan. 23, 2017, 10:33 a.m.,

Review Request 55847: AMBARI-19657: Downgrade button does not work after restart Ambari server when upgrade wizard was left open

2017-01-23 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55847/ --- Review request for Ambari and Sangeeta Ravindran. Bugs: AMBARI-19657

Re: Review Request 55789: Deleted configuration groups are displayed along with active group with is_current value set to true using Ambari API's

2017-01-23 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55789/#review162649 --- Please add Nate Cole for the review. - Di Li On Jan. 20, 2017,

Re: Review Request 55845: Storm service check failed after HOU to HDP-2.5.3 due to CNF for StormAtlasHook

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

Re: Review Request 55789: Deleted configuration groups are displayed along with active group with is_current value set to true using Ambari API's

2017-01-23 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55789/#review162648 --- Ship it! Ship It! - Di Li On Jan. 20, 2017, 11:49 p.m.,

Review Request 55848: AMBARI-19630: Ambari should accept stack version in format of x.x.x.x without the build level digits

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

Review Request 55843: Trailing slash (/) on cluster resource causes incorrect authorization logic flow

2017-01-23 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55843/ --- Review request for Ambari, Attila Magyar, Eugene Chekanskiy, Jonathan Hurley,

Re: Review Request 55843: Trailing slash (/) on cluster resource causes incorrect authorization logic flow

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

Re: Review Request 55842: Regenerate Keytabs action does not set cluster-env/security_enabled to true

2017-01-23 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55842/#review162633 --- Fix it, then Ship it! Ship It!

Re: Review Request 55845: Storm service check failed after HOU to HDP-2.5.3 due to CNF for StormAtlasHook

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

Re: Review Request 55843: Trailing slash (/) on cluster resource causes incorrect authorization logic flow

2017-01-23 Thread Attila Magyar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55843/#review162639 --- Ship it! Ship It! - Attila Magyar On Jan. 23, 2017, 2:49

Review Request 55845: Storm service check failed after HOU to HDP-2.5.3 due to CNF for StormAtlasHook

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

Re: Review Request 55842: Regenerate Keytabs action does not set cluster-env/security_enabled to true

2017-01-23 Thread Robert Levas
> On Jan. 23, 2017, 9:39 a.m., Dmitro Lisnichenko wrote: > > ambari-server/src/main/java/org/apache/ambari/server/serveraction/kerberos/UpdateKerberosConfigsServerAction.java, > > line 127 > > > > > > I think there

Re: Review Request 55842: Regenerate Keytabs action does not set cluster-env/security_enabled to true

2017-01-23 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55842/#review162629 ---

Re: Review Request 55844: ZKFC start failed

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

Re: Review Request 55845: Storm service check failed after HOU to HDP-2.5.3 due to CNF for StormAtlasHook

2017-01-23 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55845/#review162636 --- Fix it, then Ship it! Ship It!

Re: Review Request 55845: Storm service check failed after HOU to HDP-2.5.3 due to CNF for StormAtlasHook

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

Re: Review Request 55845: Storm service check failed after HOU to HDP-2.5.3 due to CNF for StormAtlasHook

2017-01-23 Thread Jonathan Hurley
> On Jan. 23, 2017, 10:30 a.m., Robert Levas wrote: > > ambari-common/src/main/python/resource_management/libraries/functions/setup_atlas_hook.py, > > line 172 > > > > > > Couldn't this use the `stack_root`

Re: Review Request 55680: On secure NN HA clusters ZKFC connects to zookeeper securely

2017-01-23 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55680/#review162637 --- Ship it! Ship It! - Robert Levas On Jan. 20, 2017, 6:21

Re: Review Request 55851: Dangling config-group hostmapping entries result in partial loading of config groups

2017-01-23 Thread Myroslav Papirkovskyy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55851/#review162672 --- Ship it! Ship It! - Myroslav Papirkovskyy On Січ. 23, 2017,

Re: Review Request 55859: Agent registration fails as local OS is not compatible with primary OS family

2017-01-23 Thread Vitalyi Brodetskyi
> On Січ. 23, 2017, 10:26 після полудня, Alejandro Fernandez wrote: > > ambari-server/conf/unix/create-python-wrap.sh, line 35 > > > > > > Why is this all of a sudden failing now? > > Weren't we testing on Ubuntu

Re: Review Request 55833: Supporting zookeeper security only from HDP 2.6

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

Review Request 55853: AMBARI-19681: Credential store should add hadoop credential provider path property to all affected configuration types

2017-01-23 Thread Nahappan Somasundaram
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55853/ --- Review request for Ambari, Miklos Gergely and Sumit Mohanty. Bugs:

Re: Review Request 55859: Agent registration fails as local OS is not compatible with primary OS family

2017-01-23 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55859/#review162720 --- ambari-server/conf/unix/create-python-wrap.sh (line 1)

Re: Review Request 55505: AMBARI-19524 : Use stack-level Atlas conf directory instead of hard-coded /etc/atlas/conf path

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

Review Request 55854: Small fixes for common log rotation

2017-01-23 Thread Madhuvanthi Radhakrishnan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55854/ --- Review request for Ambari, Alejandro Fernandez, Jayush Luniya, and Sumit

Re: Review Request 55854: Small fixes for common log rotation

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

Re: Review Request 55680: On secure NN HA clusters ZKFC connects to zookeeper securely

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

Re: Review Request 55859: Agent registration fails as local OS is not compatible with primary OS family

2017-01-23 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55859/#review162718 --- ambari-server/conf/unix/create-python-wrap.sh (line 35)

Re: Review Request 55865: Stack advisor needs to enforce component dependency for slaves and masters

2017-01-23 Thread Jaimin Jetly
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55865/ --- (Updated Jan. 24, 2017, 12:15 a.m.) Review request for Ambari, Jayush Luniya,

Review Request 55865: Stack advisor needs to enforce component dependency for slaves and masters

2017-01-23 Thread Jaimin Jetly
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55865/ --- Review request for Ambari, Jayush Luniya, Sumit Mohanty, and Yusaku Sako.

Re: Review Request 55865: Stack advisor needs to enforce component dependency for slaves and masters

2017-01-23 Thread Jaimin Jetly
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55865/ --- (Updated Jan. 24, 2017, 12:16 a.m.) Review request for Ambari, Jayush Luniya,

Re: Review Request 55609: Add Livy session recovery configurations in Ambari

2017-01-23 Thread Saisai Shao
> On Jan. 19, 2017, 4:59 a.m., Bikas Saha wrote: > > Should the URL for recovery be something like > > /apps/{livy|spark}/livy-recovery? Having /livy-recovery as a top level HDFS > > dir seems distracting. > > Saisai Shao wrote: > It will not be in the top level HDFS dirs. It will be in

Re: Review Request 55609: Add Livy session recovery configurations in Ambari

2017-01-23 Thread Saisai Shao
> On Jan. 19, 2017, 4:59 a.m., Bikas Saha wrote: > > Should the URL for recovery be something like > > /apps/{livy|spark}/livy-recovery? Having /livy-recovery as a top level HDFS > > dir seems distracting. > > Saisai Shao wrote: > It will not be in the top level HDFS dirs. It will be in

Re: Review Request 55871: Error on Files view-SIMPLE authentication is not enabled

2017-01-23 Thread Nitiraj Rathore
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55871/#review162783 --- Ship it! Ship It! - Nitiraj Rathore On Jan. 24, 2017, 5:47

Re: Review Request 55854: Small fixes for common log rotation

2017-01-23 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55854/#review162763 --- Ship it! Ship It! - Jayush Luniya On Jan. 23, 2017, 8 p.m.,

Re: Review Request 55839: Hive View 2.0: Code hints for query editor are not working

2017-01-23 Thread Nitiraj Rathore
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55839/#review162778 --- Fix it, then Ship it! Ship It!

Re: Review Request 55814: Add log rotation settings - handle upgrade scenario - Storm, Hive

2017-01-23 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55814/#review162764 --- Fix it, then Ship it!

Re: Review Request 55871: Error on Files view-SIMPLE authentication is not enabled

2017-01-23 Thread DIPAYAN BHOWMICK
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55871/#review162772 --- Ship it! Ship It! - DIPAYAN BHOWMICK On Jan. 24, 2017, 5:47

Re: Review Request 55837: Hive View 2.0: Incorrect behaviour on select of database

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

Re: Review Request 55814: Add log rotation settings - handle upgrade scenario - Storm, Hive

2017-01-23 Thread Madhuvanthi Radhakrishnan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55814/ --- (Updated Jan. 24, 2017, 6:42 a.m.) Review request for Ambari, Alejandro

Review Request 55871: Error on Files view-SIMPLE authentication is not enabled

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

Re: Review Request 55836: Hive View 2.0: Editor should be stretchable by dragging

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

Review Request 55850: Confirmation for adding host component from service summary page should be displayed once

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

Re: Review Request 55850: Confirmation for adding host component from service summary page should be displayed once

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

Re: Review Request 55812: Error during Alert: Unable to authenticate through LDAP for Hiveserver2 (also floods HS2 log with error messages)

2017-01-23 Thread Swapan Shridhar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55812/#review162662 --- Ship it! Ship It! - Swapan Shridhar On Jan. 21, 2017, 1:58

Re: Review Request 55774: AMBARI-19636: Provide default values for Kafka nofile and nproc limit properties

2017-01-23 Thread Tim Thorpe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55774/#review162651 --- Ship it! Ship It! - Tim Thorpe On Jan. 20, 2017, 4:01 p.m.,

Re: Review Request 55789: Deleted configuration groups are displayed along with active group with is_current value set to true using Ambari API's

2017-01-23 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55789/#review162659 --- There should be a test (and/or a negative test) covering this

Re: Review Request 55586: Cannot change group/user privileges in list view if the user has VIEW.USER privilege

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

Re: Review Request 55848: AMBARI-19630: Ambari should accept stack version in format of x.x.x.x without the build level digits

2017-01-23 Thread Tim Thorpe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55848/#review162654 --- Ship it! Ship It! - Tim Thorpe On Jan. 23, 2017, 4:51 p.m.,

Re: Review Request 55814: Add log rotation settings - handle upgrade scenario - Storm, Hive

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

Re: Review Request 55847: AMBARI-19657: Downgrade button does not work after restart Ambari server when upgrade wizard was left open

2017-01-23 Thread Tim Thorpe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55847/#review162667 --- Ship it! Ship It! - Tim Thorpe On Jan. 23, 2017, 6:05 p.m.,

Re: Review Request 55821: Log Search: support credential store api (part 1)

2017-01-23 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55821/#review162668 --- Ship it! Ship It! - Robert Nettleton On Jan. 22, 2017, 6:57

Review Request 55851: Dangling config-group hostmapping entries result in partial loading of config groups

2017-01-23 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55851/ --- Review request for Ambari, Myroslav Papirkovskyy and Sumit Mohanty. Bugs:

Re: Review Request 55847: AMBARI-19657: Downgrade button does not work after restart Ambari server when upgrade wizard was left open

2017-01-23 Thread Sangeeta Ravindran
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55847/#review162666 --- Ship it! Ship It! - Sangeeta Ravindran On Jan. 23, 2017,