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

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

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

2017-02-21 Thread venkat sairam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56922/#review166329 --- Ship it! Ship It! - venkat sairam On Feb. 22, 2017, 7:24 a.m

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

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

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

2017-02-21 Thread Nitiraj Rathore
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56915/#review166327 --- contrib/views/wfmanager/src/main/resources/ui/app/components/work

Re: Review Request 56918: the item of "reblance hdfs" should be hidden in NameNode menu

2017-02-21 Thread zhangxiaolu zhangxiaolu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56918/ --- (Updated 二月 22, 2017, 7:05 a.m.) Review request for Ambari, Alejandro Fernandez

Review Request 56918: the item of "reblance hdfs" should be hidden in NameNode menu

2017-02-21 Thread zhangxiaolu zhangxiaolu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56918/ --- Review request for Ambari, Alejandro Fernandez, Dmytro Sen, JaimejucoummaUW Jaim

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

2017-02-21 Thread Padma Priya N
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56915/ --- (Updated Feb. 22, 2017, 6:32 a.m.) Review request for Ambari, belliraj hb, Gaur

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

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

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

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

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

2017-02-21 Thread yao lei
> On 二月 14, 2017, 3:50 a.m., Matt wrote: > > ambari-web/test/mappers/stack_service_mapper_test.js, line 169 > > > > > > Can we have a test case where ```coHost```s are defined both ways? > > > > ``` > >

Re: Review Request 56903: AMBARI-20103 : Reduce AMS HBase metrics through whitelist.

2017-02-21 Thread Aravindan Vijayan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56903/ --- (Updated Feb. 22, 2017, 12:05 a.m.) Review request for Ambari, Dmytro Sen, Sumi

Re: Review Request 56903: AMBARI-20103 : Reduce AMS HBase metrics through whitelist.

2017-02-21 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56903/#review166264 --- Fix it, then Ship it! ambari-metrics/ambari-metrics-timelinese

Review Request 56903: AMBARI-20103 : Reduce AMS HBase metrics through whitelist.

2017-02-21 Thread Aravindan Vijayan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56903/ --- Review request for Ambari, Dmytro Sen, Sumit Mohanty, and Sid Wagle. Bugs: AMBA

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

2017-02-21 Thread Aravindan Vijayan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56857/#review166202 --- Dmitry, this patch adds the recommendation for an extra metrics co

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

2017-02-21 Thread Di Li
> On Feb. 14, 2017, 8 p.m., Di Li wrote: > > Ship It! pushed to trunk as https://git-wip-us.apache.org/repos/asf?p=ambari.git;a=commit;h=9e5c57e3e7ae4a8b3df51efcbddcdb97a1f7b398 Yuanbo, please close this rb request. - Di --- This is a

Re: Review Request 56859: Add Upgrade Logic For Removal of clusterconfigmapping

2017-02-21 Thread Jonathan Hurley
> On Feb. 21, 2017, 1:06 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog300.java, > > line 213 > > > > > > Sould first check !dbAccessor.tableHasColum

Re: Review Request 56859: Add Upgrade Logic For Removal of clusterconfigmapping

2017-02-21 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56859/#review166196 --- ambari-server/src/main/java/org/apache/ambari/server/upgrade/Upgr

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

2017-02-21 Thread Amruta Borkar
> On Feb. 21, 2017, 5:41 p.m., Sandor Magyari wrote: > > Ship It! Thank you Sandor, I don't have commit rights on trunk. Could you please help me push this to trunk? - Amruta --- This is an automatically generated e-mail. To reply, vis

Re: Review Request 56759: AMBARI-20047 : Repository config user needs to set customized service user in Ranger when service plugin is enabled.

2017-02-21 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56759/#review166192 --- ambari-server/src/main/resources/stacks/HDP/2.6/services/HDFS/con

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

2017-02-21 Thread Sandor Magyari
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56627/#review166193 --- Ship it! Ship It! - Sandor Magyari On Feb. 20, 2017, 7:04 p.

Re: Review Request 56802: AMBARI-20072. Add heap dump option for tez-site config 'tez.am.launch.cmd-opts'.

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

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

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

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

2017-02-21 Thread Prabhjyot Singh
> On Feb. 21, 2017, 5:21 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/KNOX/0.5.0.2.2/configuration/gateway-site.xml, > > line 69 > > > > > > Is this really meant to be added o

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

2017-02-21 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56604/#review166188 --- ambari-server/src/main/resources/common-services/KNOX/0.5.0.2.2/c

Re: Review Request 56884: Hive 2: Visual Explain - Remove children in the JSON dump

2017-02-21 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56884/#review166186 --- Ship it! Ship It! - Gaurav Nagar On Feb. 21, 2017, 1:09 p.m.

Re: Review Request 56859: Add Upgrade Logic For Removal of clusterconfigmapping

2017-02-21 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56859/#review166185 --- Ship it! Ship It! - Robert Levas On Feb. 20, 2017, 5:10 p.m.

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-21 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56790/ --- (Updated Feb. 21, 2017, 6:29 p.m.) Review request for Ambari and Nate Cole. C

Re: Review Request 56893: Configs filter is not reset if configs has no filter input

2017-02-21 Thread Andriy Babiichuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56893/#review166180 --- Ship it! Ship It! - Andriy Babiichuk On Лют. 21, 2017, 4:06

Review Request 56893: Configs filter is not reset if configs has no filter input

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

Re: Review Request 56800: Finalize Operations stage fails when Enabling Kerberos using the manual option

2017-02-21 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56800/#review166179 --- Ship it! Ship It! - Sebastian Toader On Feb. 17, 2017, 9:52

Re: Review Request 56891: Password fields are not disabled when should be

2017-02-21 Thread Denys Buzhor
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56891/#review166178 --- Ship it! Ship It! - Denys Buzhor On Feb. 21, 2017, 3:59 p.m.

Review Request 56891: Password fields are not disabled when should be

2017-02-21 Thread Oleg Nechiporenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56891/ --- Review request for Ambari. Bugs: AMBARI-20099 https://issues.apache.org/jir

Re: Review Request 56883: Log Search: only update schema file for Solr if it has any new fields in the local file

2017-02-21 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56883/#review166175 --- Ship it! Ship It! - Robert Nettleton On Feb. 21, 2017, 1:23

Re: Review Request 56800: Finalize Operations stage fails when Enabling Kerberos using the manual option

2017-02-21 Thread Eugene Chekanskiy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56800/#review166174 --- Ship it! Ship It! - Eugene Chekanskiy On Лют. 17, 2017, 8:52

Re: Review Request 56869: Log Search should handle turned off Credential Store too

2017-02-21 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56869/#review166173 --- Ship it! Ship It! - Robert Nettleton On Feb. 21, 2017, 11:13

Re: Review Request 56855: Grafana Dashboard for Log Search

2017-02-21 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56855/#review166172 --- Ship it! Ship It! - Robert Nettleton On Feb. 20, 2017, 1:46

Re: Review Request 56884: Hive 2: Visual Explain - Remove children in the JSON dump

2017-02-21 Thread Abhishek Kumar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56884/#review166171 --- Ship it! Ship It! - Abhishek Kumar On Feb. 21, 2017, 1:09 p.

Re: Review Request 56882: Unique Name not created automatically when designing workflow

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

Re: Review Request 56884: Hive 2: Visual Explain - Remove children in the JSON dump

2017-02-21 Thread venkat sairam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56884/#review166168 --- Ship it! Ship It! - venkat sairam On Feb. 21, 2017, 1:09 p.m

Re: Review Request 56883: Log Search: only update schema file for Solr if it has any new fields in the local file

2017-02-21 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56883/ --- (Updated Feb. 21, 2017, 1:23 p.m.) Review request for Ambari, Miklos Gergely an

Re: Review Request 56883: Log Search: only update schema file for Solr if it has any new fields in the local file

2017-02-21 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56883/#review166162 --- Fix it, then Ship it! Ship It! ambari-logsearch/ambari-logsea

Re: Review Request 56855: Grafana Dashboard for Log Search

2017-02-21 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56855/#review166161 --- Ship it! Ship It! - Oliver Szabo On Feb. 20, 2017, 1:46 p.m.

Re: Review Request 56884: Hive 2: Visual Explain - Remove children in the JSON dump

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

Re: Review Request 56884: Hive2: Visual Explain -Layout glitches of overlapping edges

2017-02-21 Thread Abhishek Kumar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56884/#review166160 --- Ship it! Ship It! - Abhishek Kumar On Feb. 21, 2017, 1:05 p.

Review Request 56884: Hive2: Visual Explain -Layout glitches of overlapping edges

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

Review Request 56883: Log Search: only update schema file for Solr if it has any new fields in the local file

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

Re: Review Request 56870: HiveView2.0: Add new UDF is not working

2017-02-21 Thread Nitiraj Rathore
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56870/#review166158 --- Ship it! Ship It! - Nitiraj Rathore On Feb. 21, 2017, 11:27

Re: Review Request 56882: Unique Name not created automatically when designing workflow

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

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

2017-02-21 Thread Renjith Kamath
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56880/#review166156 --- Ship it! Ship It! - Renjith Kamath On Feb. 21, 2017, 12:13 p

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

2017-02-21 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56783/#review166155 --- Ship it! Ship It! - Sebastian Toader On Feb. 21, 2017, 1:03

Re: Review Request 56882: Unique Name not created automatically when designing workflow

2017-02-21 Thread belliraj hb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56882/#review166154 --- Ship it! Ship It! - belliraj hb On Feb. 21, 2017, 12:17 p.m.

Review Request 56882: Unique Name not created automatically when designing workflow

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

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

2017-02-21 Thread belliraj hb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56881/ --- Review request for Ambari, DIPAYAN BHOWMICK, Gaurav Nagar, and Pallav Kulshresht

Re: Review Request 56849: AMBARI-20080.WFM does not issue confirmation message when workflow is suspended or killed

2017-02-21 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56849/#review166153 --- Ship it! Ship It! - Gaurav Nagar On Feb. 20, 2017, 11:45 a.m

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

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

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

2017-02-21 Thread Attila Magyar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56783/ --- (Updated Feb. 21, 2017, 12:03 p.m.) Review request for Ambari, Laszlo Puskas, R

Re: Review Request 56870: HiveView2.0: Add new UDF is not working

2017-02-21 Thread venkat sairam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56870/#review166150 --- Ship it! Ship It! - venkat sairam On Feb. 21, 2017, 11:27 a.

Re: Review Request 56869: Log Search should handle turned off Credential Store too

2017-02-21 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56869/#review166149 --- Ship it! Ship It! - Oliver Szabo On Feb. 21, 2017, 11:13 a.m

Re: Review Request 56869: Log Search should handle turned off Credential Store too

2017-02-21 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56869/ --- (Updated Feb. 21, 2017, 11:13 a.m.) Review request for Ambari, Oliver Szabo and

Re: Review Request 56870: HiveView2.0: Add new UDF is not working

2017-02-21 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56870/#review166141 --- Ship it! Ship It! - Gaurav Nagar On Feb. 21, 2017, 9:49 a.m.

Re: Review Request 56871: AMBARI-20090.On submit of bundle, if any coordinator path contains variables, user should be notified about custom variables.

2017-02-21 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56871/#review166139 --- Ship it! Ship It! - Gaurav Nagar On Feb. 21, 2017, 9:56 a.m.

Re: Review Request 56863: AMBARI-20084.Cut Node feature is broken in workflow manager

2017-02-21 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56863/#review166137 --- Ship it! Ship It! - Gaurav Nagar On Feb. 21, 2017, 6:04 a.m.

Re: Review Request 56871: AMBARI-20090.On submit of bundle, if any coordinator path contains variables, user should be notified about custom variables.

2017-02-21 Thread belliraj hb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56871/#review166138 --- Ship it! Ship It! - belliraj hb On Feb. 21, 2017, 9:56 a.m.,

Review Request 56871: AMBARI-20090.On submit of bundle, if any coordinator path contains variables, user should be notified about custom variables.

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

Re: Review Request 56859: Add Upgrade Logic For Removal of clusterconfigmapping

2017-02-21 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56859/#review166136 --- Ship it! Ship It! - Dmitro Lisnichenko On Feb. 21, 2017, 12:

Review Request 56870: HiveView2.0: Add new UDF is not working

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

Re: Review Request 56848: AMBARI-20079.Unable to filter by job id

2017-02-21 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56848/#review166135 --- Ship it! Ship It! - Gaurav Nagar On Feb. 20, 2017, 11:27 a.m

Re: Review Request 56846: AMBARI-20078.When uploading a file using the WFM, need a message to say that either the file upload succeeded or failed

2017-02-21 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56846/#review166134 --- Ship it! Ship It! - Gaurav Nagar On Feb. 20, 2017, 10:31 a.m

Review Request 56869: Log Search should handle turned off Credential Store too

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

Re: Review Request 56863: AMBARI-20084.Cut Node feature is broken in workflow manager

2017-02-21 Thread belliraj hb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56863/#review166129 --- Ship it! Ship It! - belliraj hb On Feb. 21, 2017, 6:04 a.m.,