Re: Review Request 50660: YARN restart icon appeared after long time after deleting ZK server

2016-08-02 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50660/#review144484 ---

Re: Review Request 50698: Ambari infra service check fails in non-root

2016-08-02 Thread Andrew Onischuk
> On Aug. 2, 2016, 12:33 p.m., Oliver Szabo wrote: > > Ship It! > > > > thanks for this patch np :) - Andrew --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50698/#review144482

Re: Review Request 50698: Ambari infra service check fails in non-root

2016-08-02 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50698/#review144481 --- Ship it! Ship It! - Dmitro Lisnichenko On Aug. 2, 2016,

Re: Review Request 50645: Ambari upgrade to 2.4.0 adds several config properties that are not necessary and some are incorrect

2016-08-02 Thread Mugdha Varadkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50645/#review144480 ---

Re: Review Request 50660: YARN restart icon appeared after long time after deleting ZK server

2016-08-02 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50660/#review144477 --- Ship it! Ship It! - Dmytro Sen On Авг. 1, 2016, 4:16 п.п.,

Review Request 50698: Ambari infra service check fails in non-root

2016-08-02 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50698/ --- Review request for Ambari and Dmitro Lisnichenko. Bugs: AMBARI-17989

Re: Review Request 50645: Ambari upgrade to 2.4.0 adds several config properties that are not necessary and some are incorrect

2016-08-02 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50645/#review144474 --- Ship it! Ship It! - Dmitro Lisnichenko On Aug. 2, 2016,

Re: Review Request 50698: Ambari infra service check fails in non-root

2016-08-02 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50698/#review144482 --- Ship it! Ship It! thanks for this patch - Oliver Szabo On

Re: Review Request 50660: YARN restart icon appeared after long time after deleting ZK server

2016-08-02 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50660/#review144476 --- Ship it! Ship It! - Andrew Onischuk On Aug. 1, 2016, 4:16

Re: Review Request 50660: YARN restart icon appeared after long time after deleting ZK server

2016-08-02 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50660/#review144475 --- Ship it! Ship It! - Dmitro Lisnichenko On Aug. 1, 2016,

Re: Review Request 50653: AMBARI-17308 Ambari Logfeeder outputs a lot of errors due to parse date

2016-08-02 Thread Oliver Szabo
> On Aug. 2, 2016, 4:58 a.m., Sebastian Toader wrote: > > ambari-server/src/main/java/org/apache/ambari/server/audit/AuditLoggerDefaultImpl.java, > > line 51 > > > > > > The 'X' stands for the ISO timezone

Re: Review Request 50660: YARN restart icon appeared after long time after deleting ZK server

2016-08-02 Thread Eugene Chekanskiy
> On Aug. 2, 2016, 1:02 p.m., Jonathan Hurley wrote: > > ambari-server/src/main/java/org/apache/ambari/server/state/ConfigHelper.java, > > lines 1124-1136 > > > > > > Previously, the existing SCH would be used to

Re: Review Request 50653: AMBARI-17308 Ambari Logfeeder outputs a lot of errors due to parse date

2016-08-02 Thread Masahiro Tanaka
> On Aug. 2, 2016, 1:58 p.m., Sebastian Toader wrote: > > ambari-server/src/main/java/org/apache/ambari/server/audit/AuditLoggerDefaultImpl.java, > > line 51 > > > > > > The 'X' stands for the ISO timezone

Re: Review Request 48766: MySQL service status needs to be more robust

2016-08-02 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48766/#review144569 ---

Re: Review Request 50725: Kerberos identity definitions in Kerberos descriptors should explicitly declare a reference

2016-08-02 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50725/ --- (Updated Aug. 2, 2016, 8:57 p.m.) Review request for Ambari, Alejandro

Re: Review Request 50653: AMBARI-17308 Ambari Logfeeder outputs a lot of errors due to parse date

2016-08-02 Thread Masahiro Tanaka
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50653/ --- (Updated Aug. 3, 2016, 10:15 a.m.) Review request for Ambari, Daniel Gergely,

Re: Review Request 49935: AMBARI-17633 yarn.nodemanager.remote-app-log-dir should be added stickybit.

2016-08-02 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49935/#review144573 --- Ship it! Ship It! - Alejandro Fernandez On Aug. 2, 2016,

Re: Review Request 50653: AMBARI-17308 Ambari Logfeeder outputs a lot of errors due to parse date

2016-08-02 Thread Masahiro Tanaka
> On 8月 2, 2016, 1:58 p.m., Sebastian Toader wrote: > > ambari-server/src/main/java/org/apache/ambari/server/audit/AuditLoggerDefaultImpl.java, > > line 51 > > > > > > The 'X' stands for the ISO timezone

Re: Review Request 50725: Kerberos identity definitions in Kerberos descriptors should explicitly declare a reference

2016-08-02 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50725/#review144574 --- Ship it! Ship It! - Alejandro Fernandez On Aug. 3, 2016,

Re: Review Request 50645: Ambari upgrade to 2.4.0 adds several config properties that are not necessary and some are incorrect

2016-08-02 Thread Sumit Mohanty
> On Aug. 2, 2016, noon, Mugdha Varadkar wrote: > > ambari-server/src/main/resources/common-services/RANGER_KMS/0.5.0.2.3/configuration/ranger-kms-audit.xml, > > line 21 > > > > > > This config file is a part of

Re: Review Request 50645: Ambari upgrade to 2.4.0 adds several config properties that are not necessary and some are incorrect

2016-08-02 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50645/#review144571 --- Ship it! Ship It! - Alejandro Fernandez On Aug. 2, 2016,

Review Request 50710: Ambari agent unable to register with server when server response is too big

2016-08-02 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50710/ --- Review request for Ambari, Sid Wagle and Vitalyi Brodetskyi. Bugs:

Review Request 50735: Data can not be migrated from Hive 1.0 to Hive2

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

Review Request 50725: Kerberos identity definitions in Kerberos descriptors should explicitly declare a reference

2016-08-02 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50725/ --- Review request for Ambari, Alejandro Fernandez, Jonathan Hurley, Nate Cole, and

Re: Review Request 50711: AMBARI-17992: Ambari should remove all references to commons-httpclient

2016-08-02 Thread Nahappan Somasundaram
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50711/ --- (Updated Aug. 2, 2016, 3:49 p.m.) Review request for Ambari, Mahadev Konar,

Review Request 50731: HAWQ service advisor shows wrong recommendations on edge cases

2016-08-02 Thread Matt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50731/ --- Review request for Ambari, Alexander Denissov and Lav Jain. Bugs: AMBARI-17996

Re: Review Request 50645: Ambari upgrade to 2.4.0 adds several config properties that are not necessary and some are incorrect

2016-08-02 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50645/ --- (Updated Aug. 3, 2016, 1:44 a.m.) Review request for Ambari, Alejandro

Review Request 50711: AMBARI-17992: Ambari should remove all references to commons-httpclient

2016-08-02 Thread Nahappan Somasundaram
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50711/ --- Review request for Ambari, Mahadev Konar, Robert Levas, Sumit Mohanty, and Sid

Re: Review Request 50711: AMBARI-17992: Ambari should remove all references to commons-httpclient

2016-08-02 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50711/#review144505 --- Ship it! Ship It! - Sumit Mohanty On Aug. 2, 2016, 4:25

Re: Review Request 50711: AMBARI-17992: Ambari should remove all references to commons-httpclient

2016-08-02 Thread Nahappan Somasundaram
> On Aug. 2, 2016, 10:49 a.m., Robert Levas wrote: > > ambari-funtest/pom.xml, lines 334-337 > > > > > > Can you remove other instances of commons-httpclient in the Ambari > > server pom.xml files > > > >

Re: Review Request 50595: AMBARI-17955: Enabled download of berkeley DB jar file to be provisioned during install

2016-08-02 Thread Venkat Ranganathan
> On Aug. 2, 2016, 1:58 a.m., Vitalyi Brodetskyi wrote: > > ambari-server/src/main/resources/common-services/FALCON/0.5.0.2.1/package/scripts/falcon.py, > > lines 200-217 > > > > > > I don't see here any condition,

Re: Review Request 50711: AMBARI-17992: Ambari should remove all references to commons-httpclient

2016-08-02 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50711/#review144509 --- Fix it, then Ship it! ambari-funtest/pom.xml

Re: Review Request 48766: MySQL service status needs to be more robust

2016-08-02 Thread Juanjo Marron
> On June 16, 2016, 10:52 p.m., Andrew Onischuk wrote: > > ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/mysql_service.py, > > line 63 > > > > > > Else should go into for, not if

Re: Review Request 50711: AMBARI-17992: Ambari should remove all references to commons-httpclient

2016-08-02 Thread Robert Levas
> On Aug. 2, 2016, 1:49 p.m., Robert Levas wrote: > > ambari-funtest/pom.xml, lines 334-337 > > > > > > Can you remove other instances of commons-httpclient in the Ambari > > server pom.xml files > > > > -

Re: Review Request 50711: AMBARI-17992: Ambari should remove all references to commons-httpclient

2016-08-02 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50711/#review144527 --- Ship it! Ship It! - Robert Levas On Aug. 2, 2016, 12:25

Re: Review Request 50691: AMBARI-17213: Worfklow designer initial contrib view

2016-08-02 Thread Venkat Ranganathan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50691/ --- (Updated Aug. 2, 2016, 12:45 a.m.) Review request for Ambari. Changes

Re: Review Request 50653: AMBARI-17308 Ambari Logfeeder outputs a lot of errors due to parse date

2016-08-02 Thread Masahiro Tanaka
> On Aug. 2, 2016, 1:58 p.m., Sebastian Toader wrote: > > ambari-server/src/main/java/org/apache/ambari/server/audit/AuditLoggerDefaultImpl.java, > > line 51 > > > > > > The 'X' stands for the ISO timezone

Review Request 50691: AMBARI-17213: Worfklow designer initial contrib view

2016-08-02 Thread Venkat Ranganathan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50691/ --- Review request for Ambari. Bugs: AMBARI-17213

Re: Review Request 50710: Ambari agent unable to register with server when server response is too big

2016-08-02 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50710/#review144497 --- Ship it! Please add jaimin as a revier as well.

Re: Review Request 50594: Fix Spark hdp.version issues in upgrading and fresh install

2016-08-02 Thread Saisai Shao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50594/ --- (Updated Aug. 2, 2016, 8:37 a.m.) Review request for Ambari, Jayush Luniya and

Re: Review Request 50653: AMBARI-17308 Ambari Logfeeder outputs a lot of errors due to parse date

2016-08-02 Thread Sebastian Toader
> On Aug. 2, 2016, 6:58 a.m., Sebastian Toader wrote: > > ambari-server/src/main/java/org/apache/ambari/server/audit/AuditLoggerDefaultImpl.java, > > line 51 > > > > > > The 'X' stands for the ISO timezone

Re: Review Request 50653: AMBARI-17308 Ambari Logfeeder outputs a lot of errors due to parse date

2016-08-02 Thread Sebastian Toader
> On Aug. 2, 2016, 6:58 a.m., Sebastian Toader wrote: > > ambari-server/src/main/java/org/apache/ambari/server/audit/AuditLoggerDefaultImpl.java, > > line 51 > > > > > > The 'X' stands for the ISO timezone

Re: Review Request 50595: AMBARI-17955: Enabled download of berkeley DB jar file to be provisioned during install

2016-08-02 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50595/#review144468 ---

Re: Review Request 50595: AMBARI-17955: Enabled download of berkeley DB jar file to be provisioned during install

2016-08-02 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50595/#review144469 ---

Re: Review Request 50595: AMBARI-17955: Enabled download of berkeley DB jar file to be provisioned during install

2016-08-02 Thread Vitalyi Brodetskyi
> On Сер. 2, 2016, 8:58 до полудня, Vitalyi Brodetskyi wrote: > > ambari-server/src/main/resources/common-services/FALCON/0.5.0.2.1/package/scripts/falcon.py, > > lines 200-217 > > > > > > I don't see here any