Re: Review Request 50777: AMBARI-18014: PXF service definition is missing pxf-json profile

2016-08-03 Thread Lav Jain
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50777/#review144711 --- Ship it! Ship It! - Lav Jain On Aug. 4, 2016, 1:24 a.m.,

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

2016-08-03 Thread Lav Jain
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50731/#review144709 --- Ship it! Ship It! - Lav Jain On Aug. 4, 2016, 1:11 a.m.,

Review Request 50777: AMBARI-18014: PXF service definition is missing pxf-json profile

2016-08-03 Thread Alexander Denissov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50777/ --- Review request for Ambari, Alejandro Fernandez, Jayush Luniya, Lav Jain, and

Re: Review Request 50666: AMBARI-10145 : Add support for tee to another Storage service?

2016-08-03 Thread Jameel Mohamed
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50666/ --- (Updated Aug. 4, 2016, 1:24 a.m.) Review request for Ambari, Aravindan

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

2016-08-03 Thread Matt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50731/ --- (Updated Aug. 3, 2016, 6:11 p.m.) Review request for Ambari, Alexander

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

2016-08-03 Thread Matt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50731/ --- (Updated Aug. 3, 2016, 5:23 p.m.) Review request for Ambari, Alexander

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

2016-08-03 Thread Matt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50731/ --- (Updated Aug. 3, 2016, 5:21 p.m.) Review request for Ambari, Alexander

Re: Review Request 50773: HiveHook fails to post messages to kafka due to missing keytab config in /etc/hive/conf/atlas-application.properties in kerberized cluster

2016-08-03 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50773/#review144695 --- Ship it! Ship It! - Robert Levas On Aug. 3, 2016, 7:46

Review Request 50773: HiveHook fails to post messages to kafka due to missing keytab config in /etc/hive/conf/atlas-application.properties in kerberized cluster

2016-08-03 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50773/ --- Review request for Ambari, Robert Levas, Sumit Mohanty, and Swapan Shridhar.

Re: Review Request 50666: AMBARI-10145 : Add support for tee to another Storage service?

2016-08-03 Thread Jameel Mohamed
> On Aug. 3, 2016, 9:31 p.m., Sid Wagle wrote: > > ambari-metrics/ambari-metrics-timelineservice/src/main/java/org/apache/hadoop/yarn/server/applicationhistoryservice/metrics/timeline/TimelineMetricsAggregatorSink.java, > > line 40 > >

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

2016-08-03 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 50653: AMBARI-17308 Ambari Logfeeder outputs a lot of errors due to parse date

2016-08-03 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50653/#review144684 --- Thank you for the contribution. Merged into trunk and

Re: Review Request 50666: AMBARI-10145 : Add support for tee to another Storage service?

2016-08-03 Thread Jameel Mohamed
> On Aug. 3, 2016, 9:31 p.m., Sid Wagle wrote: > > ambari-metrics/ambari-metrics-timelineservice/src/main/java/org/apache/hadoop/yarn/server/applicationhistoryservice/metrics/timeline/TimelineMetricsAggregatorSink.java, > > line 40 > >

Review Request 50450: API for bulk delete hostcomponent

2016-08-03 Thread Ajit Kumar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50450/ --- Review request for Ambari, Alejandro Fernandez, Zhe (Joe) Wang, Nate Cole, and

Re: Review Request 50666: AMBARI-10145 : Add support for tee to another Storage service?

2016-08-03 Thread Sid Wagle
> On Aug. 3, 2016, 9:28 p.m., Sid Wagle wrote: > > ambari-metrics/ambari-metrics-timelineservice/src/test/java/org/apache/hadoop/yarn/server/applicationhistoryservice/metrics/timeline/TimelineMetricsAggregatorMemorySink.java, > > line 33 > >

Re: Review Request 50666: AMBARI-10145 : Add support for tee to another Storage service?

2016-08-03 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50666/#review144673 ---

Re: Review Request 50545: Ambari should not recursively chown for HAWQ hdfs upon every start

2016-08-03 Thread Lav Jain
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50545/ --- (Updated Aug. 3, 2016, 9:28 p.m.) Review request for Ambari, Alexander

Re: Review Request 50666: AMBARI-10145 : Add support for tee to another Storage service?

2016-08-03 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50666/#review144671 ---

Re: Review Request 50545: Ambari should not recursively chown for HAWQ hdfs upon every start

2016-08-03 Thread Lav Jain
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50545/ --- (Updated Aug. 3, 2016, 9:18 p.m.) Review request for Ambari, Alexander

Re: Review Request 50047: Kafka listeners property does not show SASL_PLAINTEXT protocol when Kerberos is enabled

2016-08-03 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50047/#review144670 --- Ship it! Ship It! - Robert Levas On Aug. 3, 2016, 4:47

Re: Review Request 50047: Kafka listeners property does not show SASL_PLAINTEXT protocol when Kerberos is enabled

2016-08-03 Thread Anita Jebaraj
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50047/ --- (Updated Aug. 3, 2016, 8:47 p.m.) Review request for Ambari, Alejandro

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

2016-08-03 Thread Alexander Denissov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50731/#review144642 ---

Re: Review Request 50666: AMBARI-10145 : Add support for tee to another Storage service?

2016-08-03 Thread Jameel Mohamed
> On Aug. 3, 2016, 6:20 p.m., Aravindan Vijayan wrote: > > ambari-metrics/ambari-metrics-timelineservice/src/main/java/org/apache/hadoop/yarn/server/applicationhistoryservice/metrics/timeline/TimelineMetricsAggregatorSink.java, > > line 37 > >

Re: Review Request 50666: AMBARI-10145 : Add support for tee to another Storage service?

2016-08-03 Thread Aravindan Vijayan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50666/#review144637 ---

Re: Review Request 50666: AMBARI-10145 : Add support for tee to another Storage service?

2016-08-03 Thread Jameel Mohamed
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50666/#review144636 ---

Re: Review Request 50666: AMBARI-10145 : Add support for tee to another Storage service?

2016-08-03 Thread Aravindan Vijayan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50666/#review144635 ---

Re: Review Request 50756: RU Downgrade failure while downgrading from erie to DAL-m10

2016-08-03 Thread Alejandro Fernandez
> On Aug. 3, 2016, 5:41 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/stacks/HDP/2.3/upgrades/upgrade-2.5.xml, > > line 797 > > > > > > This does not look like a correct fix. > > > >

Re: Review Request 50756: RU Downgrade failure while downgrading from erie to DAL-m10

2016-08-03 Thread Dmitro Lisnichenko
> On Aug. 3, 2016, 8:41 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/stacks/HDP/2.3/upgrades/upgrade-2.5.xml, > > line 797 > > > > > > This does not look like a correct fix. > > > >

Re: Review Request 50756: RU Downgrade failure while downgrading from erie to DAL-m10

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

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

2016-08-03 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50594/#review144625 --- Ship it! Ship It! - Jayush Luniya On Aug. 2, 2016, 8:37

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

2016-08-03 Thread Dmytro Sen
> On Авг. 2, 2016, 3:35 п.п., Sid Wagle wrote: > > ambari-server/pom.xml, line 1117 > > > > > > Where is this being used ? I think it's a views dependency. It's been there before replacing jetty 8 with jetty 9

Review Request 50756: RU Downgrade failure while downgrading from erie to DAL-m10

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

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

2016-08-03 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50710/ --- (Updated Авг. 3, 2016, 4:44 п.п.) Review request for Ambari, Jaimin Jetly, Sid

Re: Review Request 50753: Cleared cached resources from ambari-server on host removal

2016-08-03 Thread Sandor Magyari
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50753/#review144619 --- Ship it! Ship It! - Sandor Magyari On Aug. 3, 2016, 3:17

Re: Review Request 50753: Cleared cached resources from ambari-server on host removal

2016-08-03 Thread Laszlo Puskas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50753/ --- (Updated Aug. 3, 2016, 3:17 p.m.) Review request for Ambari, Robert Nettleton,

Re: Review Request 50753: Cleared cached resources from ambari-server on host removal

2016-08-03 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50753/#review144618 --- Ship it! Ship It! - Robert Nettleton On Aug. 3, 2016, 2:47

Re: Review Request 50753: Cleared cached resources from ambari-server on host removal

2016-08-03 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50753/#review144617 --- Fix it, then Ship it!

Review Request 50753: Cleared cached resources from ambari-server on host removal

2016-08-03 Thread Laszlo Puskas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50753/ --- Review request for Ambari, Robert Nettleton, Sandor Magyari, and Sebastian

Re: Review Request 50752: Tune HDFS opts parameters to trigger GC more predicitably

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

Review Request 50752: Tune HDFS opts parameters to trigger GC more predicitably

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

Re: Review Request 50742: operationStatus and taskStatus audit log should contain remoteIp message

2016-08-03 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50742/#review144605 ---

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

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

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

2016-08-03 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50725/#review144600 --- Ship it! Ship It! - Oliver Szabo On Aug. 3, 2016, 12:57

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

2016-08-03 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50735/#review144597 --- Ship it! Ship It! - Pallav Kulshreshtha On Aug. 3, 2016,

Review Request 50742: operationStatus and taskStatus audit log should contain remoteIp message

2016-08-03 Thread wang yaoxin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50742/ --- Review request for Ambari, Laszlo Puskas, Oliver Szabo, Sandor Magyari, and

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

2016-08-03 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

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

2016-08-03 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50653/#review144594 --- Fix it, then Ship it!

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

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

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

2016-08-03 Thread Andrew Onischuk
> 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 50653: AMBARI-17308 Ambari Logfeeder outputs a lot of errors due to parse date

2016-08-03 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50653/#review144587 --- Ship it! Ship It! - Sebastian Toader On Aug. 3, 2016, 3:15

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

2016-08-03 Thread DIPAYAN BHOWMICK
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50735/#review144585 --- Ship it! Ship It! - DIPAYAN BHOWMICK On Aug. 3, 2016, 4:55