Re: Review Request 56974: Services should be able to specify that credential store is always enabled

2017-02-22 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56974/ --- (Updated Feb. 23, 2017, 7:53 a.m.) Review request for Ambari, Alejandro

Review Request 56974: Services should be able to specify that credential store is always enabled

2017-02-22 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56974/ --- Review request for Ambari, Alejandro Fernandez, Mugdha Varadkar, and Sid Wagle.

Re: Review Request 56973: Hive view 2.0 : Hanging query - no cancel available

2017-02-22 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56973/#review166490 --- Ship it! Ship It! - Pallav Kulshreshtha On Feb. 23, 2017,

Re: Review Request 56973: Hive view 2.0 : Hanging query - no cancel available

2017-02-22 Thread Ashwin Rajeev
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56973/#review166491 --- Ship it! Ship It! - Ashwin Rajeev On Feb. 23, 2017, 7:13

Review Request 56973: Hive view 2.0 : Hanging query - no cancel available

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

Re: Review Request 56971: Changes/Overwrites to Bundles and Coordinators Show as Workflow, Not able to select text from worflow 'Definition' tab

2017-02-22 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56971/#review166489 --- Ship it! Ship It! - Pallav Kulshreshtha On Feb. 23, 2017,

Re: Review Request 56972: Improper action node name after importing the asset

2017-02-22 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56972/#review166488 --- Ship it! Ship It! - Pallav Kulshreshtha On Feb. 23, 2017,

Re: Review Request 56972: Improper action node name after importing the asset

2017-02-22 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56972/#review166486 --- Ship it! Ship It! - Gaurav Nagar On Feb. 23, 2017, 6:44

Re: Review Request 56971: Changes/Overwrites to Bundles and Coordinators Show as Workflow, Not able to select text from worflow 'Definition' tab

2017-02-22 Thread belliraj hb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56971/#review166485 --- Ship it! Ship It! - belliraj hb On Feb. 23, 2017, 6:30

Re: Review Request 56972: Improper action node name after importing the asset

2017-02-22 Thread belliraj hb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56972/#review166484 --- Ship it! Ship It! - belliraj hb On Feb. 23, 2017, 6:44

Review Request 56972: Improper action node name after importing the asset

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

Review Request 56971: Changes/Overwrites to Bundles and Coordinators Show as Workflow, Not able to select text from worflow 'Definition' tab

2017-02-22 Thread venkat sairam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56971/ --- Review request for Ambari, belliraj hb, DIPAYAN BHOWMICK, Gaurav Nagar, Nitiraj

Re: Review Request 56956: Remove OrderedDict python module code from ranger stack

2017-02-22 Thread Mugdha Varadkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56956/#review166481 --- Ship it! Ship It! - Mugdha Varadkar On Feb. 22, 2017, 11:16

Re: Review Request 56927: AMBARI-20116 : Stack Recommendation for Atlas not working on change of Kafka broker port.

2017-02-22 Thread Mugdha Varadkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56927/#review166480 --- Ship it! Ship It! - Mugdha Varadkar On Feb. 22, 2017, 3:34

Re: Review Request 56881: Cannot edit join node name in Workflow Manager

2017-02-22 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56881/#review166477 --- Ship it! Ship It! - Gaurav Nagar On Feb. 21, 2017, 12:15

Re: Review Request 56965: Hive 2: "insert udf" opens empty drop down

2017-02-22 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56965/#review166475 --- Ship it! Ship It! - Gaurav Nagar On Feb. 23, 2017, 3:40

Re: Review Request 56965: Hive 2: "insert udf" opens empty drop down

2017-02-22 Thread Abhishek Kumar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56965/#review166472 --- Ship it! Ship It! - Abhishek Kumar On Feb. 23, 2017, 3:40

Review Request 56965: Hive 2: "insert udf" opens empty drop down

2017-02-22 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56965/ --- Review request for Ambari, Abhishek Kumar, DIPAYAN BHOWMICK, Gaurav Nagar,

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

2017-02-22 Thread yao lei
> On 二月 22, 2017, 10:28 p.m., Jaimin Jetly wrote: > > We already hasve the stack way to co-host a component. In 3.0.0, we have > > committed a patch by which you can enforce a slave/master component to be > > co-hosted with another slave/master component. There are snapshots of how > > UI

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

2017-02-22 Thread yao lei
> On 二月 22, 2017, 9:33 p.m., Alejandro Fernandez wrote: > > Ship It! Thank you for reviewing this patch - yao --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55817/#review166421

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

2017-02-22 Thread yao lei
> On 二月 22, 2017, 9:21 p.m., Di Li wrote: > > ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/metainfo.xml, > > line 109 > > > > > > Hello Yao, > > > > As you said in comment to Matt's

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

2017-02-22 Thread Di Li
> On Feb. 22, 2017, 9:21 p.m., Di Li wrote: > > ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/metainfo.xml, > > line 109 > > > > > > Hello Yao, > > > > As you said in comment to Matt's

Re: Review Request 56961: DataNode Storage alert is duplicated

2017-02-22 Thread Xi Wang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56961/#review166464 --- Ship it! Ship It! - Xi Wang On Feb. 22, 2017, 5:02 p.m.,

Review Request 56961: DataNode Storage alert is duplicated

2017-02-22 Thread Richard Zang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56961/ --- Review request for Ambari, Alexandr Antonenko and Xi Wang. Bugs: AMBARI-20125

Re: Review Request 56857: Confusing AMS collector heap size validation loop

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

Re: Review Request 56956: Remove OrderedDict python module code from ranger stack

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

Review Request 56957: Click feedback/disable toggle is missing for transition buttons in Kerberos Wizard - prone to various failures

2017-02-22 Thread Denys Buzhor
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56957/ --- Review request for Ambari. Bugs: AMBARI-20123

Re: Review Request 56956: Remove OrderedDict python module code from ranger stack

2017-02-22 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56956/#review166442 --- Ship it! Ship It! - Miklos Gergely On Feb. 22, 2017, 11:16

Review Request 56956: Remove OrderedDict python module code from ranger stack

2017-02-22 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56956/ --- Review request for Ambari, Jonathan Hurley, Miklos Gergely, Mugdha Varadkar,

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

2017-02-22 Thread Jaimin Jetly
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55817/#review166433 --- We already hasve the stack way to co-host a component. In 3.0.0,

Re: Review Request 56857: Confusing AMS collector heap size validation loop

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

Re: Review Request 56950: Error during EU while updating Ranger Log4J service configs

2017-02-22 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56950/#review166422 ---

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

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

Re: Review Request 56943: "Use RedHat Satellite/Spacewalk" setting not working correctly in Ambari 2.4.2.0

2017-02-22 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56943/#review166419 --- Ship it! Ship It! - Jonathan Hurley On Feb. 22, 2017, 1:57

Re: Review Request 56950: Error during EU while updating Ranger Log4J service configs

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

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

2017-02-22 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55817/#review166416 ---

Review Request 56943: "Use RedHat Satellite/Spacewalk" setting not working correctly in Ambari 2.4.2.0

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

Re: Review Request 56942: Test and fix new modals styles on Ambari

2017-02-22 Thread Aleksandr Kovalenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56942/#review166386 --- Ship it! Ship It! - Aleksandr Kovalenko On Фев. 22, 2017,

Review Request 56942: Test and fix new modals styles on Ambari

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

Re: Review Request 56926: Ambari reports grafana service is down when its running causing other services to not start

2017-02-22 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56926/#review166383 --- ambari-common/src/main/python/ambari_commons/network.py (line

Re: Review Request 56783: Removing secure ACLs from Kafka znodes during dekerberization

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

Re: Review Request 56880: Zeppelin should configure hadoop.proxyuser in HDFS core-site.xml

2017-02-22 Thread DIPAYAN BHOWMICK
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56880/#review166377 --- Ship it! Ship It! - DIPAYAN BHOWMICK On Feb. 22, 2017, 5:17

Re: Review Request 56880: Zeppelin should configure hadoop.proxyuser in HDFS core-site.xml

2017-02-22 Thread Prabhjyot Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56880/ --- (Updated Feb. 22, 2017, 5:17 p.m.) Review request for Ambari, Alejandro

Review Request 56927: AMBARI-20116 : Stack Recommendation for Atlas not working on change of Kafka broker port.

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

Re: Review Request 56881: Cannot edit join node name in Workflow Manager

2017-02-22 Thread belliraj hb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56881/#review166365 --- Ship it! Ship It! - belliraj hb On Feb. 21, 2017, 12:15

Re: Review Request 56925: Hive2: Visual Explain - Eliminate "limit -1"

2017-02-22 Thread Abhishek Kumar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56925/#review166364 --- Ship it! Ship It! - Abhishek Kumar On Feb. 22, 2017, 2:12

Review Request 56926: Ambari reports grafana service is down when its running causing other services to not start

2017-02-22 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56926/ --- Review request for Ambari, Aravindan Vijayan, Sid Wagle, and Vitalyi Brodetskyi.

Re: Review Request 56925: Hive2: Visual Explain - Eliminate "limit -1"

2017-02-22 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56925/ --- (Updated Feb. 22, 2017, 2:12 p.m.) Review request for Ambari, Abhishek Kumar,

Review Request 56925: Hive2: Visual Explain - Eliminate "limit -1"

2017-02-22 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56925/ --- Review request for Ambari, Abhishek Kumar, DIPAYAN BHOWMICK, Gaurav Nagar,

Re: Review Request 56661: HiveView2.0: Table list under DB panel on Query tab appears only if clicked on db name and Not clear how to set the database

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

Re: Review Request 56661: HiveView2.0: Table list under DB panel on Query tab appears only if clicked on db name and Not clear how to set the database

2017-02-22 Thread venkat sairam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56661/ --- (Updated Feb. 22, 2017, 12:01 p.m.) Review request for Ambari, DIPAYAN

Re: Review Request 56857: Confusing AMS collector heap size validation loop

2017-02-22 Thread Dmytro Sen
> On Фев. 21, 2017, 7:04 п.п., Aravindan Vijayan wrote: > > Dmitry, this patch adds the recommendation for an extra metrics collector > > if the number of sinks > 999. > > > > Does this solve the problem of stack advisor recommending progressively > > higher heap sizes for a single collector

Re: Review Request 56915: AMBARI-20107.Action node shows unsupported properties even though there are none in workflow manager.

2017-02-22 Thread Nitiraj Rathore
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56915/#review166355 --- Ship it! Ship It! - Nitiraj Rathore On Feb. 22, 2017, 6:32

Review Request 56923: Add polyfills for Array.includes and String.includes

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

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

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

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

2017-02-22 Thread venkat sairam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56606/ --- (Updated Feb. 22, 2017, 11:14 a.m.) Review request for Ambari, DIPAYAN

Re: Review Request 56790: Getting Internal Server Error (500) on services API while trying to start all services with atleast one component in INSTALL_FAILED state

2017-02-22 Thread Dmytro Grinenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56790/#review166351 --- Ship it! Ship It! - Dmytro Grinenko On Feb. 22, 2017, 10:51

Re: Review Request 56922: Hive2: Query in query editor is getting modified while navigating across different tabs

2017-02-22 Thread Nitiraj Rathore
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56922/#review166341 --- Ship it! Ship It! - Nitiraj Rathore On Feb. 22, 2017, 7:24

Re: Review Request 56915: AMBARI-20107.Action node shows unsupported properties even though there are none in workflow manager.

2017-02-22 Thread belliraj hb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56915/#review166338 --- Ship it! Ship It! - belliraj hb On Feb. 22, 2017, 6:32