Re: Review Request 49640: Identify config changes added to Ambari-2.4.0 and mark them to not get added during Ambari upgrade

2016-07-06 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49640/#review141126 --- ambari-server/src/main/resources/stacks/HDP/2.0.6/configuration/c

Re: Review Request 49727: AMBARI-17598 Permission mismatch b/w 'Cluster user' and 'read only user' from older ambari

2016-07-06 Thread Yusaku Sako
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49727/#review141110 --- Ship it! Ship It! - Yusaku Sako On July 6, 2016, 9:27 p.m.,

Re: Review Request 49711: AMBARI-17593: Ambari server backup error - failure if backup size exceeds 4GB

2016-07-06 Thread Ajit Kumar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49711/#review141107 --- Ship it! Ship It! - Ajit Kumar On July 6, 2016, 4:51 p.m., N

Re: Review Request 49735: AMBARI-17570 Lack of importing ClientComponentHasNoStatus

2016-07-06 Thread Juanjo Marron
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49735/#review141106 --- Ship it! Ship It! - Juanjo Marron On July 6, 2016, 11:45 p.

Re: Review Request 48309: AMBARI-17047: Firewall check returns WARNING even if iptables and firewalld are stopped on CentOS7

2016-07-06 Thread Masahiro Tanaka
> On June 13, 2016, 1:53 p.m., Andrew Onischuk wrote: > > Ship It! > > Masahiro Tanaka wrote: > Thank you! > > Masahiro Tanaka wrote: > Could you commit it? > > Andrew Onischuk wrote: > Done. Please close the reviewboard now. > > Andrew Onischuk wrote: > reverted the patch, ap

Re: Review Request 48309: AMBARI-17047: Firewall check returns WARNING even if iptables and firewalld are stopped on CentOS7

2016-07-06 Thread Masahiro Tanaka
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48309/ --- (Updated July 6, 2016, 11:47 p.m.) Review request for Ambari, Andrew Onischuk,

Review Request 49735: AMBARI-17570 Lack of importing ClientComponentHasNoStatus

2016-07-06 Thread Masahiro Tanaka
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49735/ --- Review request for Ambari, Alejandro Fernandez, Jayush Luniya, and Juanjo Marro

Re: Review Request 49734: fix spark.driver.extraLibraryPath to include native gpl library

2016-07-06 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49734/#review141100 --- Ship it! Ship It! - Alejandro Fernandez On July 6, 2016, 11:

Review Request 49734: fix spark.driver.extraLibraryPath to include native gpl library

2016-07-06 Thread Weiqing Yang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49734/ --- Review request for Ambari, Sumit Mohanty and Srimanth Gunturi. Bugs: AMBARI-175

Re: Review Request 48973: AMBARI-17324. kafka should set zookeeper.set.acl to true when kerberos enabled

2016-07-06 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48973/#review141091 --- Pushed to trunk, commit 73111e81dd7b78ab458ae8d147eeedce0cb61d1a b

Re: Review Request 48973: AMBARI-17324. kafka should set zookeeper.set.acl to true when kerberos enabled

2016-07-06 Thread Sriharsha Chintalapani
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48973/ --- (Updated July 6, 2016, 9:27 p.m.) Review request for Ambari and Alejandro Ferna

Re: Review Request 48973: AMBARI-17324. kafka should set zookeeper.set.acl to true when kerberos enabled

2016-07-06 Thread Sriharsha Chintalapani
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48973/ --- (Updated July 6, 2016, 9:27 p.m.) Review request for Ambari and Alejandro Ferna

Review Request 49727: AMBARI-17598 Permission mismatch b/w 'Cluster user' and 'read only user' from older ambari

2016-07-06 Thread Zhe (Joe) Wang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49727/ --- Review request for Ambari, Jaimin Jetly, Richard Zang, Vivek Ratnavel Subramania

Re: Review Request 48973: AMBARI-17234. kafka should set zookeeper.set.acl to true when kerberos enabled

2016-07-06 Thread Sriharsha Chintalapani
> On July 5, 2016, 7:32 p.m., Robert Levas wrote: > > Ship It! is this merged? - Sriharsha --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48973/#review140857

Re: Review Request 49676: Add atlas-application config sections to all services that run Atlas hook, e.g., Hive, Falcon, Storm, Sqoop

2016-07-06 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49676/ --- (Updated July 6, 2016, 8:29 p.m.) Review request for Ambari, Madhan Neethiraj,

Re: Review Request 49640: Identify config changes added to Ambari-2.4.0 and mark them to not get added during Ambari upgrade

2016-07-06 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49640/#review141064 --- ambari-server/src/main/resources/common-services/HAWQ/2.0.0/confi

Review Request 49721: AMBARI-17580: Unable to add MASTER component on all nodes

2016-07-06 Thread Shantanu Mundkur
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49721/ --- Review request for Ambari and Jaimin Jetly. Bugs: AMBARI-17580 https://issu

Re: Review Request 49676: Add atlas-application config sections to all services that run Atlas hook, e.g., Hive, Falcon, Storm, Sqoop

2016-07-06 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49676/#review141060 --- Ship it! Ship It! - Robert Levas On July 6, 2016, 2:48 p.m.,

Re: Review Request 48972: AMBARI-17253 Ambari Alert causes too many wanings in ZooKeeper logs.

2016-07-06 Thread Nate Cole
On July 4, 2016, 6:58 p.m., Masahiro Tanaka wrote: > > What about existing clusters? We probably need to modify any existing ZK > > alerts with this using the UpgradeCatalog. > > Masahiro Tanaka wrote: > Thank you for reviewing. Which one should we change, > `UpgradeCatalog230.java`, or `U

Re: Review Request 49676: Add atlas-application config sections to all services that run Atlas hook, e.g., Hive, Falcon, Storm, Sqoop

2016-07-06 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49676/#review141057 --- Ship it! Ship It! - Nate Cole On July 6, 2016, 2:48 p.m., Al

Re: Review Request 49676: Add atlas-application config sections to all services that run Atlas hook, e.g., Hive, Falcon, Storm, Sqoop

2016-07-06 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49676/ --- (Updated July 6, 2016, 6:48 p.m.) Review request for Ambari, Madhan Neethiraj,

Re: Review Request 49665: authorizer.class.name not being set on secure kafka clusters

2016-07-06 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49665/#review141039 --- Ship it! Ship It! - Alejandro Fernandez On July 6, 2016, 12:

Re: Review Request 49590: While changing NN, DN directories from UI, proper warning should be present for invalid values

2016-07-06 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49590/#review141037 --- Ship it! Ship It! - Alejandro Fernandez On July 6, 2016, 1:4

Review Request 49711: AMBARI-17593: Ambari server backup error - failure if backup size exceeds 4GB

2016-07-06 Thread Nahappan Somasundaram
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49711/ --- Review request for Ambari, Ajit Kumar and Sumit Mohanty. Bugs: AMBARI-17593

Re: Review Request 49385: Hive and Oozie db displayed incorrectly on the installer review page

2016-07-06 Thread Alexandr Antonenko
> On July 6, 2016, 1:52 p.m., Alexandr Antonenko wrote: > > Ship It! > > Sangeeta Ravindran wrote: > Thanks Alexandr. Can you help push the fix. done - Alexandr --- This is an automatically generated e-mail. To reply, visit: https:

Re: Review Request 49637: Zeppelin: service install failure on Suse due to bash error

2016-07-06 Thread Rohit Choudhary
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49637/#review141016 --- Ship it! Ship It! - Rohit Choudhary On July 5, 2016, 2:14 p.

Re: Review Request 49385: Hive and Oozie db displayed incorrectly on the installer review page

2016-07-06 Thread Sangeeta Ravindran
> On July 5, 2016, 4:11 p.m., Andrii Tkach wrote: > > Ship It! Thanks Andrii. - Sangeeta --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49385/#review140804 --

Re: Review Request 49385: Hive and Oozie db displayed incorrectly on the installer review page

2016-07-06 Thread Sangeeta Ravindran
> On July 6, 2016, 1:52 p.m., Alexandr Antonenko wrote: > > Ship It! Thanks Alexandr. Can you help push the fix. - Sangeeta --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49385/#review140987

Re: Review Request 49590: While changing NN, DN directories from UI, proper warning should be present for invalid values

2016-07-06 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49590/#review140993 --- Ship it! Ship It! - Dmytro Sen On Июль 6, 2016, 1:43 п.п., A

Re: Review Request 48309: AMBARI-17047: Firewall check returns WARNING even if iptables and firewalld are stopped on CentOS7

2016-07-06 Thread Andrew Onischuk
> On June 13, 2016, 1:53 p.m., Andrew Onischuk wrote: > > Ship It! > > Masahiro Tanaka wrote: > Thank you! > > Masahiro Tanaka wrote: > Could you commit it? > > Andrew Onischuk wrote: > Done. Please close the reviewboard now. reverted the patch, apache jira - Andrew --

Re: Review Request 49637: Zeppelin: service install failure on Suse due to bash error

2016-07-06 Thread Prabhjyot Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49637/#review140992 --- Ship it! manually tested on suse11. - Prabhjyot Singh On Jul

Re: Review Request 48309: AMBARI-17047: Firewall check returns WARNING even if iptables and firewalld are stopped on CentOS7

2016-07-06 Thread Andrew Onischuk
> On June 13, 2016, 1:53 p.m., Andrew Onischuk wrote: > > Ship It! > > Masahiro Tanaka wrote: > Thank you! > > Masahiro Tanaka wrote: > Could you commit it? Done. Please close the reviewboard now. - Andrew --- This is an autom

Re: Review Request 49385: Hive and Oozie db displayed incorrectly on the installer review page

2016-07-06 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49385/#review140987 --- Ship it! Ship It! - Alexandr Antonenko On June 30, 2016, 8:5

Re: Review Request 49590: While changing NN, DN directories from UI, proper warning should be present for invalid values

2016-07-06 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49590/ --- (Updated July 6, 2016, 1:43 p.m.) Review request for Ambari, Alejandro Fernande

Re: Review Request 49676: Add atlas-application config sections to all services that run Atlas hook, e.g., Hive, Falcon, Storm, Sqoop

2016-07-06 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49676/#review140986 --- Ship it! ambari-common/src/main/python/resource_management/lib

Re: Review Request 49701: Log search does not show Livy logs

2016-07-06 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49701/ --- (Updated July 6, 2016, 1:28 p.m.) Review request for Ambari, Oliver Szabo, Robe

Re: Review Request 49438: Editable solrconfig.xml for Logsearch (+ atlas)

2016-07-06 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49438/ --- (Updated July 6, 2016, 1:25 p.m.) Review request for Ambari, Alejandro Fernande

Re: Review Request 49590: While changing NN, DN directories from UI, proper warning should be present for invalid values

2016-07-06 Thread Andrew Onischuk
> On July 5, 2016, 7:39 p.m., Sid Wagle wrote: > > ambari-server/src/main/resources/stacks/HDP/2.0.6/services/stack_advisor.py, > > line 1476 > > > > > > Based on this impl, we would get 1 warning at a time vs gett

Re: Review Request 49665: authorizer.class.name not being set on secure kafka clusters

2016-07-06 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49665/ --- (Updated July 6, 2016, 8:41 a.m.) Review request for Ambari, Alejandro Fernande

Re: Review Request 49590: While changing NN, DN directories from UI, proper warning should be present for invalid values

2016-07-06 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49590/ --- (Updated July 6, 2016, 12:40 p.m.) Review request for Ambari, Alejandro Fernand

Review Request 49703: Add SmartSense activty logs to Log Search

2016-07-06 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49703/ --- Review request for Ambari, Oliver Szabo, Robert Nettleton, and Sumit Mohanty. B

Re: Review Request 49703: Add SmartSense activty logs to Log Search

2016-07-06 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49703/#review140981 --- Ship it! Ship It! - Oliver Szabo On July 6, 2016, 12:28 p.m.

Re: Review Request 49702: HIVE_SERVER_INTERACTIVE STOP failed with error "Python script has been killed due to timeout after waiting 900 secs"

2016-07-06 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49702/#review140979 --- Ship it! Ship It! - Dmytro Sen On Июль 6, 2016, 11:37 д.п.,

Re: Review Request 49701: Log search does not show Livy logs

2016-07-06 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49701/#review140978 --- Ship it! Ship It! - Oliver Szabo On July 6, 2016, 11:39 a.m.

Review Request 49701: Log search does not show Livy logs

2016-07-06 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49701/ --- Review request for Ambari, Oliver Szabo, Robert Nettleton, and Sumit Mohanty. B

Re: Review Request 49702: HIVE_SERVER_INTERACTIVE STOP failed with error "Python script has been killed due to timeout after waiting 900 secs"

2016-07-06 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49702/#review140976 --- ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/

Re: Review Request 49702: HIVE_SERVER_INTERACTIVE STOP failed with error "Python script has been killed due to timeout after waiting 900 secs"

2016-07-06 Thread Andrew Onischuk
> On July 6, 2016, 11:38 a.m., Andrew Onischuk wrote: > > ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/hive_server_interactive.py, > > line 234 > > > > > > Remove since all calls

Review Request 49702: HIVE_SERVER_INTERACTIVE STOP failed with error "Python script has been killed due to timeout after waiting 900 secs"

2016-07-06 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49702/ --- Review request for Ambari and Vitalyi Brodetskyi. Bugs: AMBARI-17582 https:

Re: Review Request 49635: Enable simulating logfeeder inputs

2016-07-06 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49635/ --- (Updated July 6, 2016, 10:26 a.m.) Review request for Ambari, Oliver Szabo, Rob

Re: Review Request 49535: Ambari Agent memory Leak fix.

2016-07-06 Thread Andrew Onischuk
> On July 5, 2016, 1:22 p.m., Jonathan Hurley wrote: > > I don't see how this actually fixes the problem; importing gc from > > subprocess would be the same as importing it normally, no? There isn't a > > different gc module that subprocess uses. Also, after you patch it, you > > then re-impor