Review Request 57718: Duplicate entries in DB for auto__instance privileges upon Ambari server restart

2017-03-16 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57718/ --- Review request for Ambari, Attila Magyar, Balázs Bence Sári, DIPAYAN BHOWMICK,

Re: Review Request 57716: Ambari View examples phone-list-upgrade-view/phone-list-view cannot display normally

2017-03-16 Thread yao lei
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57716/ --- (Updated 三月 17, 2017, 1:47 a.m.) Review request for Ambari, Alejandro

Review Request 57716: Ambari View examples phone-list-upgrade-view/phone-list-view cannot display normally

2017-03-16 Thread yao lei
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57716/ --- Review request for Ambari, DIPAYAN BHOWMICK and Tom Beerbower. Bugs:

Re: Review Request 57703: Add a new property to yarn-site.xml yarn.log.server.web-service.url

2017-03-16 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57703/#review169232 --- Ship it! Ship It! - Alejandro Fernandez On March 17, 2017,

Re: Review Request 57703: Add a new property to yarn-site.xml yarn.log.server.web-service.url

2017-03-16 Thread Madhuvanthi Radhakrishnan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57703/ --- (Updated March 17, 2017, 12:24 a.m.) Review request for Ambari, Alejandro

Re: Review Request 57692: HDP 3.0 support for Spark with configs, kerberos, widgets, metrics, quicklinks, and themes

2017-03-16 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57692/#review169226 --- Fix it, then Ship it!

Re: Review Request 57703: Add a new property to yarn-site.xml yarn.log.server.web-service.url

2017-03-16 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57703/#review169224 ---

Review Request 57709: Yarn Queue manager capacity field allows entering invalid values

2017-03-16 Thread Anita Jebaraj
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57709/ --- Review request for Ambari, belliraj hb, Di Li, DIPAYAN BHOWMICK, and Sangeeta

Re: Review Request 57704: HDP 3.0 support for Tez with configs, kerberos, widgets, metrics, quicklinks, and themes

2017-03-16 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57704/#review169218 --- Fix it, then Ship it! Fix it and ship it, thanks!

Re: Review Request 57625: Minor refactoring and clean up in ambari-server

2017-03-16 Thread Yusaku Sako
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57625/#review169216 --- Ship it! Ship It! - Yusaku Sako On March 16, 2017, 2:14

Review Request 57704: HDP 3.0 support for Tez with configs, kerberos, widgets, metrics, quicklinks, and themes

2017-03-16 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57704/ --- Review request for Ambari, Alejandro Fernandez and Dmytro Sen. Bugs:

Re: Review Request 57656: Side Nav: several issues and adjustments

2017-03-16 Thread Richard Zang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57656/#review169213 --- Ship it! Ship It! - Richard Zang On March 15, 2017, 6:37

Re: Review Request 57703: Add a new property to yarn-site.xml yarn.log.server.web-service.url

2017-03-16 Thread Madhuvanthi Radhakrishnan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57703/ --- (Updated March 16, 2017, 9:08 p.m.) Review request for Ambari, Alejandro

Review Request 57703: Add a new property to yarn-site.xml yarn.log.server.web-service.url

2017-03-16 Thread Madhuvanthi Radhakrishnan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57703/ --- Review request for Ambari, Alejandro Fernandez and Sumit Mohanty. Bugs:

Re: Review Request 57507: Diff tool shows additional line when comparing configs from custom config group

2017-03-16 Thread Jaimin Jetly
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57507/#review169207 --- Ship it! If there is no UX impact and patch has been tested to

Re: Review Request 57507: Diff tool shows additional line when comparing configs from custom config group

2017-03-16 Thread Jaimin Jetly
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57507/#review169206 --- ambari-web/app/mixins/common/configs/configs_comparator.js Lines

Re: Review Request 57687: AMBARI-20443 No need to show `hive_ambari_database`

2017-03-16 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57687/#review169205 ---

Re: Review Request 57610: Tokenize kerberos principal name appearing in kerberos rules in exported blueprint

2017-03-16 Thread Robert Levas
> On March 16, 2017, 9:12 a.m., Sandor Magyari wrote: > > The question is here whether is useful or not to export auth_to_local > > properties, since they are generated at deploy time anyway. May be would be > > better to exclude from export. > > Could you please also add Robert Levas as a

Re: Review Request 57610: Tokenize kerberos principal name appearing in kerberos rules in exported blueprint

2017-03-16 Thread Amruta Borkar
> On March 16, 2017, 1:12 p.m., Sandor Magyari wrote: > > The question is here whether is useful or not to export auth_to_local > > properties, since they are generated at deploy time anyway. May be would be > > better to exclude from export. > > Could you please also add Robert Levas as a

Re: Review Request 57672: Add a new property to yarn-site.xml yarn.log.server.web-service.url

2017-03-16 Thread Madhuvanthi Radhakrishnan
> On March 16, 2017, 3:01 p.m., Robert Nettleton wrote: > > Patch looks fine to me, although it would be good to add a unit test for > > the change to the Blueprint config processor. > > > > I'd also recommend one additional manual test: Deploy a Blueprint with the > > default configuration

Re: Review Request 57610: Tokenize kerberos principal name appearing in kerberos rules in exported blueprint

2017-03-16 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57610/#review169191 ---

Re: Review Request 57683: Accessing a user after sync with AD is failing with 500

2017-03-16 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57683/#review169190 --- Ship it! Ship It! - Robert Levas On March 16, 2017, 12:40

Re: Review Request 57698: Incorrect position of services actions dropdown after scrolling down the sidebar

2017-03-16 Thread Aleksandr Kovalenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57698/#review169181 --- Ship it! Ship It! - Aleksandr Kovalenko On Март 16, 2017,

Review Request 57698: Incorrect position of services actions dropdown after scrolling down the sidebar

2017-03-16 Thread Andrii Babiichuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57698/ --- Review request for Ambari and Aleksandr Kovalenko. Bugs: AMBARI-20483

Re: Review Request 57683: Accessing a user after sync with AD is failing with 500

2017-03-16 Thread Balázs Bence Sári
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57683/ --- (Updated March 16, 2017, 4:40 p.m.) Review request for Ambari, Attila

Re: Review Request 57610: Tokenize kerberos principal name appearing in kerberos rules in exported blueprint

2017-03-16 Thread Robert Levas
> On March 16, 2017, 9:12 a.m., Sandor Magyari wrote: > > The question is here whether is useful or not to export auth_to_local > > properties, since they are generated at deploy time anyway. May be would be > > better to exclude from export. > > Could you please also add Robert Levas as a

Review Request 57692: HDP 3.0 support for Spark with configs, kerberos, widgets, metrics, quicklinks, and themes

2017-03-16 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57692/ --- Review request for Ambari, Alejandro Fernandez, Sid Wagle, and Vitalyi

Re: Review Request 57686: AMBARI-20472 MapRed History server start does not create directories in HDFS when WEBHDFS is disabled.

2017-03-16 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57686/ --- (Updated March 16, 2017, 3:55 p.m.) Review request for Ambari and Alejandro

Re: Review Request 57686: AMBARI-20472 MapRed History server start does not create directories in HDFS when WEBHDFS is disabled.

2017-03-16 Thread Di Li
> On March 16, 2017, 1:25 p.m., Attila Doroszlai wrote: > > The bug is also present in > > [branch-2.5](http://git-wip-us.apache.org/repos/asf?p=ambari.git=commit=ae75c871e497ee4a8de23579b3fbc175f8d073d5), > > shouldn't the fix be committed there, too? yes, sure, I will push the fix to both

Re: Review Request 57683: Accessing a user after sync with AD is failing with 500

2017-03-16 Thread Sandor Magyari
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57683/#review169165 --- Ship it! Ship It! - Sandor Magyari On March 16, 2017, 1:16

Re: Review Request 57582: Create a prototype of ambari-server swagger integration

2017-03-16 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57582/#review169161 --- "As a result of which currently as a temporary workaround, all

Re: Review Request 57672: Add a new property to yarn-site.xml yarn.log.server.web-service.url

2017-03-16 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57672/#review169159 --- Fix it, then Ship it! Patch looks fine to me, although it

Re: Review Request 57378: RU: MR2 service check failed during rolling upgrade

2017-03-16 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57378/#review169157 --- Ship it! Ship It! - Jonathan Hurley On March 7, 2017, 10:56

Re: Review Request 57684: Ubuntu: When Disable Security Infra Solr Instance and RM start are failed

2017-03-16 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57684/#review169154 --- Ship it! Ship It! - Sumit Mohanty On March 16, 2017, 11:41

Re: Review Request 57688: Start/Stop Service command may silently fail without any reporting for user

2017-03-16 Thread Denys Buzhor
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57688/#review169153 --- Ship it! Ship It! - Denys Buzhor On March 16, 2017, 1 p.m.,

Re: Review Request 57625: Minor refactoring and clean up in ambari-server

2017-03-16 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57625/ --- (Updated Березень 16, 2017, 2:14 після полудня) Review request for Ambari,

Re: Review Request 57589: Duplicate only global notifications of origin alert group when duplicating alert group in the the dialog of Manage Alert Groups

2017-03-16 Thread Nate Cole
> On March 15, 2017, 11:23 a.m., Nate Cole wrote: > > LGTM, make sure you get a UI person to take a look. > > wang jianfei wrote: > OK, thanks very much! By the way, can you recommend some UI persons for > me? Possibly jaimin? - Nate

Re: Review Request 57583: HDFS Bytes Written and HDFS Bytes Read maybe not correct

2017-03-16 Thread Nate Cole
> On March 15, 2017, 11:24 a.m., Nate Cole wrote: > > LGTM, make sure a UI person reviews. > > zhangxiaolu zhangxiaolu wrote: > Yes, of course.Could you provide some UI person for me? I'm not sure who > are experts.I just know you and yusaku.Thanks a lot. You have them already on this

Re: Review Request 57686: AMBARI-20472 MapRed History server start does not create directories in HDFS when WEBHDFS is disabled.

2017-03-16 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57686/#review169144 --- Ship it! The bug is also present in

Re: Review Request 57685: Security status commands fill ambari-agent.log with stack traces

2017-03-16 Thread Eugene Chekanskiy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57685/#review169145 --- Ship it! Ship It! - Eugene Chekanskiy On Березень 16, 2017,

Re: Review Request 57683: Accessing a user after sync with AD is failing with 500

2017-03-16 Thread Balázs Bence Sári
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57683/ --- (Updated March 16, 2017, 1:16 p.m.) Review request for Ambari, Attila

Re: Review Request 57610: Tokenize kerberos principal name appearing in kerberos rules in exported blueprint

2017-03-16 Thread Sandor Magyari
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57610/#review169142 --- The question is here whether is useful or not to export

Review Request 57688: Start/Stop Service command may silently fail without any reporting for user

2017-03-16 Thread Oleg Nechiporenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57688/ --- Review request for Ambari and Denys Buzhor. Bugs: ambari-20481

Review Request 57687: AMBARI-20443 No need to show `hive_ambari_database`

2017-03-16 Thread Masahiro Tanaka
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57687/ --- Review request for Ambari, Alejandro Fernandez, Jayush Luniya, and Yusaku Sako.

Re: Review Request 57507: Diff tool shows additional line when comparing configs from custom config group

2017-03-16 Thread Oleg Nechiporenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57507/ --- (Updated March 16, 2017, 12:49 p.m.) Review request for Ambari and Jaimin

Review Request 57686: AMBARI-20472 MapRed History server start does not create directories in HDFS when WEBHDFS is disabled.

2017-03-16 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57686/ --- Review request for Ambari and Alejandro Fernandez. Bugs: AMBARI-20472

Review Request 57685: Security status commands fill ambari-agent.log with stack traces

2017-03-16 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57685/ --- Review request for Ambari and Vitalyi Brodetskyi. Bugs: AMBARI-20480

Re: Review Request 57683: Accessing a user after sync with AD is failing with 500

2017-03-16 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57683/#review169132 --- Ship it! Ship It! - Sebastian Toader On March 16, 2017,

Re: Review Request 57684: Ubuntu: When Disable Security Infra Solr Instance and RM start are failed

2017-03-16 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57684/#review169131 --- Ship it! Ship It! - Dmytro Sen On Март 16, 2017, 11:41

Re: Review Request 57557: AMBARI-20417. Spark2 Thrift Server cannot be started after upgrade due to incompletely uploaded tar.gz

2017-03-16 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57557/#review169128 --- Ship it! Ship It! - Sebastian Toader On March 16, 2017,

Re: Review Request 57684: Ubuntu: When Disable Security Infra Solr Instance and RM start are failed

2017-03-16 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57684/#review169127 --- Ship it! Ship It! - Andrew Onischuk On March 16, 2017,

Review Request 57684: Ubuntu: When Disable Security Infra Solr Instance and RM start are failed

2017-03-16 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57684/ --- Review request for Ambari, Andrew Onischuk, Dmytro Sen, and Sumit Mohanty.

Re: Review Request 57682: Workflow manager is only showing 10 actions for a workflow in dashboard

2017-03-16 Thread belliraj hb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57682/#review169124 --- Ship it! Ship It! - belliraj hb On March 16, 2017, 11:10

Review Request 57683: Accessing a user after sync with AD is failing with 500

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

Review Request 57682: Workflow manager is only showing 10 actions for a workflow in dashboard

2017-03-16 Thread Madhan Reddy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57682/ --- Review request for Ambari, belliraj hb, DIPAYAN BHOWMICK, and Pallav

Re: Review Request 57681: Unable to load UI after enabling some experimental features

2017-03-16 Thread Denys Buzhor
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57681/#review169118 --- Ship it! Ship It! - Denys Buzhor On March 16, 2017, 10:19

Re: Review Request 57681: Unable to load UI after enabling some experimental features

2017-03-16 Thread Andrii Babiichuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57681/ --- (Updated Березень 16, 2017, 10:19 до полудня) Review request for Ambari and

Re: Review Request 57589: Duplicate only global notifications of origin alert group when duplicating alert group in the the dialog of Manage Alert Groups

2017-03-16 Thread wang jianfei
> On 三月 15, 2017, 3:23 p.m., Nate Cole wrote: > > LGTM, make sure you get a UI person to take a look. OK, thanks very much! By the way, can you recommend some UI persons for me? - wang --- This is an automatically generated e-mail. To

Review Request 57681: Unable to load UI after enabling some experimental features

2017-03-16 Thread Andrii Babiichuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57681/ --- Review request for Ambari and Denys Buzhor. Bugs: AMBARI-20478

Re: Review Request 57678: AMBARI-20475.Issue with cut/copy functionality in workflow manager

2017-03-16 Thread belliraj hb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57678/#review169100 --- Ship it! Ship It! - belliraj hb On March 16, 2017, 7:30

Review Request 57678: AMBARI-20475.Issue with cut/copy functionality in workflow manager

2017-03-16 Thread Padma Priya N
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57678/ --- Review request for Ambari, belliraj hb, DIPAYAN BHOWMICK, and Pallav