Re: Review Request 56604: Zeppelin UI to support proxy access via Knox in Ambari

2017-02-13 Thread Prabhjyot Singh
> On Feb. 13, 2017, 6:31 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/stacks/HDP/2.6/services/KNOX/configuration/topology.xml, > > line 146 > > > > > > What should the behavior be during a

Re: Review Request 56598: AMBARI-19985. Workflow Manager Styling changes

2017-02-13 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56598/#review165458 --- Ship it! Ship It! - Gaurav Nagar On Feb. 13, 2017, 12:41

Re: Review Request 56622: AMBARI-19993 : Update atlas log-4j.xml configurations in ambari

2017-02-13 Thread Mugdha Varadkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56622/#review165454 --- Ship it! Ship It! - Mugdha Varadkar On Feb. 13, 2017, 10:37

Review Request 56641: Change Livy recovery folder permission to 0700

2017-02-13 Thread Saisai Shao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56641/ --- Review request for Ambari, Jayush Luniya and Sumit Mohanty. Bugs:

Re: Review Request 56603: AMBARI-19953: AMS HBase RegionServer process is still alive after switching AMS to embedded mode

2017-02-13 Thread Qin Liu
> On Feb. 13, 2017, 7:39 p.m., Aravindan Vijayan wrote: > > Ship It! Thank you very much Aravindan! - Qin --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56603/#review165382

Re: Review Request 56488: Hive2: Invalid query execution not handled properly

2017-02-13 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56488/ --- (Updated Feb. 14, 2017, 4:56 a.m.) Review request for Ambari, DIPAYAN

Review Request 56638: Unable to stop service when master components is down

2017-02-13 Thread Wang Jie
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56638/ --- Review request for Ambari, Zhe (Joe) Wang, Richard Zang, and Yusaku Sako.

Re: Review Request 55817: AMBARI-19618 Make cohosted components configurable in metainfo.xml instead of hardcoding in UI

2017-02-13 Thread Matt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55817/#review165432 --- I'm curious whether this feature allow cohosting of another

Re: Review Request 56604: Zeppelin UI to support proxy access via Knox in Ambari

2017-02-13 Thread Prabhjyot Singh
> On Feb. 13, 2017, 6:31 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/stacks/HDP/2.6/services/KNOX/configuration/topology.xml, > > line 146 > > > > > > What should the behavior be during a

Re: Review Request 56636: When yarn-site.xml is changed, MapReduce related components should be required to restart

2017-02-13 Thread Yuanbo Liu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56636/ --- (Updated Feb. 14, 2017, 3 a.m.) Review request for Ambari and Di Li.

Re: Review Request 55817: AMBARI-19618 Make cohosted components configurable in metainfo.xml instead of hardcoding in UI

2017-02-13 Thread yao lei
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55817/ --- (Updated 二月 14, 2017, 1:20 a.m.) Review request for Ambari, Alejandro

Review Request 56627: Cluster deployment fails if cluster template has fewer host_groups than that in blueprint

2017-02-13 Thread Amruta Borkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56627/ --- Review request for Ambari, Di Li and Robert Nettleton. Bugs: AMBARI-19991

Re: Review Request 56231: AMBARI-19829: Several HDFS/YARN widgets on Heatmaps show N/A

2017-02-13 Thread Aravindan Vijayan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56231/#review165406 --- Ship it! Ship It! - Aravindan Vijayan On Feb. 2, 2017, 8:02

Review Request 56625: Add PowerPC architecture support to contrib views

2017-02-13 Thread Vi On
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56625/ --- Review request for Ambari. Bugs: AMBARI-19994

Review Request 56622: AMBARI-19993 : Update atlas log-4j.xml configurations in ambari

2017-02-13 Thread Vishal Suvagia
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56622/ --- Review request for Ambari, Alejandro Fernandez, Gautam Borad, Mugdha Varadkar,

Re: Review Request 56488: Hive2: xxx; never returns

2017-02-13 Thread DIPAYAN BHOWMICK
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56488/#review165390 --- contrib/views/hive20/src/main/resources/ui/app/adapters/query.js

Re: Review Request 56488: Hive2: xxx; never returns

2017-02-13 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56488/ --- (Updated Feb. 13, 2017, 7:35 p.m.) Review request for Ambari, DIPAYAN

Re: Review Request 56488: Hive2: xxx; never returns

2017-02-13 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56488/ --- (Updated Feb. 13, 2017, 7:33 p.m.) Review request for Ambari, DIPAYAN

Re: Review Request 56605: Stop All services call failed to stop flume agent

2017-02-13 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56605/#review165381 --- Ship it! Ship It! - Sumit Mohanty On Feb. 13, 2017, 5:22

Review Request 56603: AMBARI-19953: AMS HBase RegionServer process is still alive after switching AMS to embedded mode

2017-02-13 Thread Qin Liu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56603/ --- Review request for Ambari, Aravindan Vijayan, Dmytro Sen, and Sid Wagle. Bugs:

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

2017-02-13 Thread Jaimin Jetly
> On Dec. 15, 2016, 4:49 p.m., Jonathan Hurley wrote: > > ambari-server/src/main/java/org/apache/ambari/server/actionmanager/ActionDBAccessorImpl.java, > > lines 445-447 > > > > > > This could be problematic if the

Re: Review Request 56607: AMBARI-19915 Add Ranger KMS SSL properties in ambari stack

2017-02-13 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56607/#review165378 ---

Re: Review Request 56606: Allow user to view Tez View after executing query

2017-02-13 Thread DIPAYAN BHOWMICK
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56606/#review165373 --- contrib/views/hive20/src/main/resources/ui/app/adapters/query.js

Review Request 56607: AMBARI-19915 Add Ranger KMS SSL properties in ambari stack

2017-02-13 Thread Mugdha Varadkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56607/ --- Review request for Ambari, Alejandro Fernandez, Gautam Borad, Jayush Luniya,

Re: Review Request 56573: Increase default timeout for RM

2017-02-13 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56573/#review165377 ---

Re: Review Request 56606: Allow user to view Tez View after executing query

2017-02-13 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56606/#review165368 ---

Re: Review Request 56394: Provide user-warning while upgrading clusters to move certificates/keystores/truststores out of conf folder

2017-02-13 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56394/#review165366 --- Ship it! Ship It! - Alejandro Fernandez On Feb. 10, 2017,

Review Request 56606: Allow user to view Tez View after executing query

2017-02-13 Thread venkat sairam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56606/ --- Review request for Ambari, DIPAYAN BHOWMICK, Nitiraj Rathore, Pallav

Re: Review Request 56599: ambari-server upgrade is not idempotent

2017-02-13 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56599/#review165364 --- Ship it! Ship It! - Alejandro Fernandez On Feb. 13, 2017,

Re: Review Request 56604: Zeppelin UI to support proxy access via Knox in Ambari

2017-02-13 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56604/#review165363 ---

Re: Review Request 56545: AMBARI-19959. Allow skipping view extraction in ambari-server setup

2017-02-13 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56545/#review165360 --- Ship it! Ship It! - Alejandro Fernandez On Feb. 10, 2017,

Review Request 56605: Stop All services call failed to stop flume agent

2017-02-13 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56605/ --- Review request for Ambari, Alejandro Fernandez, Andrew Onischuk, Sumit Mohanty,

Review Request 56604: Zeppelin UI to support proxy access via Knox in Ambari

2017-02-13 Thread Prabhjyot Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56604/ --- Review request for Ambari, Alejandro Fernandez, DIPAYAN BHOWMICK, Gaurav Nagar,

Re: Review Request 56602: Oozie start failed after enabling credential store

2017-02-13 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56602/#review165346 --- Ship it! Ship It! - Sumit Mohanty On Feb. 13, 2017, 3:58

Review Request 56602: Oozie start failed after enabling credential store

2017-02-13 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56602/ --- Review request for Ambari, Sumit Mohanty and Sid Wagle. Bugs: AMBARI-19987

Re: Review Request 56599: ambari-server upgrade is not idempotent

2017-02-13 Thread Attila Magyar
> On Feb. 13, 2017, 3:49 p.m., Jonathan Hurley wrote: > > ambari-server/src/main/java/org/apache/ambari/server/orm/entities/PermissionEntity.java, > > lines 243-251 > > > > > > Any reason you didn't use a Set here

Re: Review Request 56599: ambari-server upgrade is not idempotent

2017-02-13 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56599/#review165341 --- Fix it, then Ship it!

Re: Review Request 56552: Ambari Server Unit Test Failures

2017-02-13 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56552/#review165339 --- Ship it! Ship It! - Robert Levas On Feb. 10, 2017, 12:41

Re: Review Request 56599: ambari-server upgrade is not idempotent

2017-02-13 Thread Laszlo Puskas
> On Feb. 13, 2017, 3:08 p.m., Laszlo Puskas wrote: > > ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog250Test.java, > > line 1672 > > > > > > Instead mocking tehe injector itself, add

Re: Review Request 56599: ambari-server upgrade is not idempotent

2017-02-13 Thread Laszlo Puskas
> On Feb. 13, 2017, 3:08 p.m., Laszlo Puskas wrote: > > ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog250Test.java, > > line 1672 > > > > > > Instead mocking tehe injector itself, add

Re: Review Request 56599: ambari-server upgrade is not idempotent

2017-02-13 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56599/#review165337 --- Ship it! Ship It! - Robert Levas On Feb. 13, 2017, 8:48

Re: Review Request 56599: ambari-server upgrade is not idempotent

2017-02-13 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56599/#review165335 --- Ship it! Ship It! - Nate Cole On Feb. 13, 2017, 8:48 a.m.,

Re: Review Request 56599: ambari-server upgrade is not idempotent

2017-02-13 Thread Attila Magyar
> On Feb. 13, 2017, 3:08 p.m., Laszlo Puskas wrote: > > ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog250Test.java, > > line 1672 > > > > > > Instead mocking tehe injector itself, add

Re: Review Request 56599: ambari-server upgrade is not idempotent

2017-02-13 Thread Laszlo Puskas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56599/#review165330 ---

Re: Review Request 56600: Default group permissions for the user home directories created by the post user creation script should be 'hdfs' instead of 'hadoop'

2017-02-13 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56600/#review165324 --- Ship it! Ship It! - Sebastian Toader On Feb. 13, 2017, 2:24

Re: Review Request 56600: Default group permissions for the user home directories created by the post user creation script should be 'hdfs' instead of 'hadoop'

2017-02-13 Thread Attila Magyar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56600/#review165323 --- Ship it! Ship It! - Attila Magyar On Feb. 13, 2017, 1:24

Re: Review Request 56600: Default group permissions for the user home directories created by the post user creation script should be 'hdfs' instead of 'hadoop'

2017-02-13 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56600/#review165322 --- Ship it! Ship It! - Attila Doroszlai On Feb. 13, 2017, 2:24

Review Request 56599: ambari-server upgrade is not idempotent

2017-02-13 Thread Attila Magyar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56599/ --- Review request for Ambari, Jonathan Hurley, Laszlo Puskas, Nate Cole, Robert

Re: Review Request 56552: Ambari Server Unit Test Failures

2017-02-13 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56552/#review165321 --- Ship it! Ship It! - Nate Cole On Feb. 10, 2017, 12:41 p.m.,

Review Request 56600: Default group permissions for the user home directories created by the post user creation script should be 'hdfs' instead of 'hadoop'

2017-02-13 Thread Laszlo Puskas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56600/ --- Review request for Ambari, Attila Doroszlai, Attila Magyar, and Sebastian

Re: Review Request 56598: AMBARI-19985. Workflow Manager Styling changes

2017-02-13 Thread belliraj hb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56598/#review165318 --- Ship it! Ship It! - belliraj hb On Feb. 13, 2017, 12:41

Review Request 56598: AMBARI-19985. Workflow Manager Styling changes

2017-02-13 Thread Madhan Reddy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56598/ --- Review request for Ambari, belliraj hb, Gaurav Nagar, Nitiraj Rathore, and

Re: Review Request 56596: Admin View: issues with Users/Groups table filters

2017-02-13 Thread Andrii Tkach
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56596/#review165312 --- Ship it! Ship It! - Andrii Tkach On Feb. 13, 2017, 11:49

Re: Review Request 56574: Perf: Deploy 3000 Agent cluster and find perf bugs. Part 5

2017-02-13 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56574/#review165311 --- Ship it! Ship It! - Dmytro Sen On Фев. 11, 2017, 10:51

Re: Review Request 56589: Hive1 View in Ambari shows wrong status

2017-02-13 Thread Rohit Choudhary
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56589/#review165310 --- Ship it! Ship It! - Rohit Choudhary On Feb. 13, 2017, 7:42

Review Request 56597: Externalize the logic for making navigation bar interactive

2017-02-13 Thread Oleg Nechiporenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56597/ --- Review request for Ambari and Jaimin Jetly. Bugs: AMBARI-19983

Review Request 56596: Admin View: issues with Users/Groups table filters

2017-02-13 Thread Andriy Babiichuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56596/ --- Review request for Ambari and Andrii Tkach. Bugs: AMBARI-19982

Re: Review Request 56589: Hive1 View in Ambari shows wrong status

2017-02-13 Thread DIPAYAN BHOWMICK
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56589/#review165306 --- Ship it! Ship It! - DIPAYAN BHOWMICK On Feb. 13, 2017, 7:42