Re: Review Request 45824: Failed to install oozie client

2016-04-06 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45824/#review127407 --- Ship it! Ship It! - Sumit Mohanty On April 6, 2016, 5:58

Re: Review Request 45169: AMBARI-15388 - Upgrade XML should be pushed down as much as possible to the services

2016-04-06 Thread Tim Thorpe
> On March 29, 2016, 12:49 p.m., Nate Cole wrote: > > I think you need a more concrete way of ordering here. What if two > > services are marked as YARN? Which one takes precedence? You may > > want to introduce an in order to > > specifically state how it happens. Order would be a

Review Request 45826: Descriptions for HAWQ sysctl params mention incorrect default values

2016-04-06 Thread Alexander Denissov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45826/ --- Review request for Ambari, bhuvnesh chaudhary, Goutam Tadi, Lav Jain, and Matt.

Re: Review Request 45810: zeppelin_log_dir change leads to fail for different Zeppelin Notebook service actions

2016-04-06 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45810/#review127382 --- Ship it! Ship It! - Alejandro Fernandez On April 6, 2016,

Re: Review Request 45826: Descriptions for HAWQ sysctl params mention incorrect default values

2016-04-06 Thread Goutam Tadi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45826/#review127388 --- Ship it! Ship It! - Goutam Tadi On April 6, 2016, 6:04

Re: Review Request 45781: AMBARI-15722 [Ambari Web] move RedHat Satellite option out of experimental

2016-04-06 Thread Nate Cole
> On April 6, 2016, 9:59 a.m., Nate Cole wrote: > > Is this hooked in anywhere? I don't see any API calls (but that's probably > > ok for this review). Any ambari-admin view changes? > > Zhe (Joe) Wang wrote: > I thought the requirement for this issue is to expose the RedHat > Satellite

Re: Review Request 45744: AMBARI-15696: BulkCommand stop NodeManagers stops the wrong component

2016-04-06 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45744/ --- (Updated April 6, 2016, 6:38 p.m.) Review request for Ambari and Alejandro

Review Request 45832: Add ignore-bad-hosts flag to hawq activate standby custom action

2016-04-06 Thread bhuvnesh chaudhary
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45832/ --- Review request for Ambari, jun aoki, Matt, and Oleksandr Diachenko. Bugs:

Review Request 45825: AMBARI-15136 Alert definitions: Percentage params are shown incrrectly

2016-04-06 Thread Zhe (Joe) Wang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45825/ --- Review request for Ambari, Alexandr Antonenko, Eugene Chekanskiy, Jaimin Jetly,

Re: Review Request 45825: AMBARI-15136 Alert definitions: Percentage params are shown incrrectly

2016-04-06 Thread Richard Zang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45825/#review127393 --- Ship it! Ship It! - Richard Zang On April 6, 2016, 6:02

Re: Review Request 45826: Descriptions for HAWQ sysctl params mention incorrect default values

2016-04-06 Thread Lav Jain
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45826/#review127405 --- Ship it! - Lav Jain On April 6, 2016, 6:04 p.m., Alexander

Re: Review Request 45169: AMBARI-15388 - Upgrade XML should be pushed down as much as possible to the services

2016-04-06 Thread Nate Cole
> On March 29, 2016, 8:49 a.m., Nate Cole wrote: > > I think you need a more concrete way of ordering here. What if two > > services are marked as YARN? Which one takes precedence? You may > > want to introduce an in order to > > specifically state how it happens. Order would be a

Re: Review Request 45826: Descriptions for HAWQ sysctl params mention incorrect default values

2016-04-06 Thread Matt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45826/#review127390 --- Ship it! Ship It! - Matt On April 6, 2016, 11:04 a.m.,

Re: Review Request 45778: AMBARI-15577: Stack Featurize HIVE service

2016-04-06 Thread Juanjo Marron
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45778/ --- (Updated April 6, 2016, 8:49 p.m.) Review request for Ambari, Alejandro

Re: Review Request 45837: Kafka jaas files are not generated on a Kafa upgrade if security.inter.broker.protocol is set to SASL_PLAINTEXT

2016-04-06 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45837/#review127464 --- Ship it! Ship It! - Alejandro Fernandez On April 6, 2016,

Re: Review Request 45835: AMBARI-15635. Storm & Kafka should've default nproc , nofile values.

2016-04-06 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45835/#review127461 ---

Re: Review Request 45837: Kafka jaas files are not generated on a Kafa upgrade if security.inter.broker.protocol is set to SASL_PLAINTEXT

2016-04-06 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45837/#review127481 ---

Re: Review Request 45778: AMBARI-15577: Stack Featurize HIVE service

2016-04-06 Thread Juanjo Marron
> On April 6, 2016, 7:04 a.m., Jayush Luniya wrote: > > ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/hive_service_interactive.py, > > line 113 > > > > > > Keep cmd as a list

Re: Review Request 45837: Kafka jaas files are not generated on a Kafa upgrade if security.inter.broker.protocol is set to SASL_PLAINTEXT

2016-04-06 Thread Sriharsha Chintalapani
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45837/ --- (Updated April 6, 2016, 9:12 p.m.) Review request for Ambari, Alejandro

Re: Review Request 45837: Kafka jaas files are not generated on a Kafa upgrade if security.inter.broker.protocol is set to SASL_PLAINTEXT

2016-04-06 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45837/#review127458 --- Ship it! Ship It! - Jayush Luniya On April 6, 2016, 9:12

Re: Review Request 45833: Webhcat Server failed to stop while stopping all the services

2016-04-06 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45833/#review127445 --- Ship it! Ship It! - Jayush Luniya On April 6, 2016, 8:21

Re: Review Request 45835: AMBARI-15635. Storm & Kafka should've default nproc , nofile values.

2016-04-06 Thread Sriharsha Chintalapani
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45835/ --- (Updated April 6, 2016, 9:12 p.m.) Review request for Ambari, Alejandro

Re: Review Request 45835: AMBARI-15635. Storm & Kafka should've default nproc , nofile values.

2016-04-06 Thread Alejandro Fernandez
> On April 6, 2016, 8:58 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/KAFKA/0.8.1.2.2/configuration/kafka-env.xml, > > line 61 > > > > > > Does this value need to change

Re: Review Request 45833: Webhcat Server failed to stop while stopping all the services

2016-04-06 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45833/ --- (Updated April 6, 2016, 10:21 p.m.) Review request for Ambari, Andrew Onischuk

Review Request 45833: Webhcat Server failed to stop while stopping all the services

2016-04-06 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45833/ --- Review request for Ambari, Andrew Onischuk and Dmytro Sen. Bugs: AMBARI-15744

Re: Review Request 45809: ambari-server script not able to load log4.properties other than the default location.

2016-04-06 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45809/ --- (Updated April 6, 2016, 10:59 a.m.) Review request for Ambari and Vitalyi

Re: Review Request 45809: ambari-server script not able to load log4.properties other than the default location.

2016-04-06 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45809/#review127309 --- Ship it! Ship It! - Vitalyi Brodetskyi On Квітень 6, 2016,

Re: Review Request 45841: Restart LLAP action should be shown only when Hive Interactive Server is present in the cluster

2016-04-06 Thread Yusaku Sako
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45841/#review127506 --- Ship it! Ship It! - Yusaku Sako On April 6, 2016, 11:25

Review Request 45845: AMBARI-15694 : AMS returns truncated results when it exceeds the metrics service default result limit config

2016-04-06 Thread Aravindan Vijayan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45845/ --- Review request for Ambari, Dmytro Sen, Sumit Mohanty, and Sid Wagle. Bugs:

Review Request 45857: RU/EU PreCheck to ensure dfs.client.retry.policy.enable is set to false

2016-04-06 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45857/ --- Review request for Ambari, Dmytro Grinenko, Dmitro Lisnichenko, Jonathan

Re: Review Request 45837: Kafka jaas files are not generated on a Kafa upgrade if security.inter.broker.protocol is set to SASL_PLAINTEXT

2016-04-06 Thread Sriharsha Chintalapani
> On April 6, 2016, 10:37 p.m., Jayush Luniya wrote: > > ambari-server/src/main/resources/common-services/KAFKA/0.8.1.2.2/package/scripts/params.py, > > line 143 > > > > > > @Sriharsha > > > > There is a

Re: Review Request 45837: Kafka jaas files are not generated on a Kafa upgrade if security.inter.broker.protocol is set to SASL_PLAINTEXT

2016-04-06 Thread Sumit Mohanty
> On April 6, 2016, 10:37 p.m., Jayush Luniya wrote: > > ambari-server/src/main/resources/common-services/KAFKA/0.8.1.2.2/package/scripts/params.py, > > line 143 > > > > > > @Sriharsha > > > > There is a

Re: Review Request 45837: Kafka jaas files are not generated on a Kafa upgrade if security.inter.broker.protocol is set to SASL_PLAINTEXT

2016-04-06 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45837/#review127524 --- Ship it! Ship It! - Sumit Mohanty On April 7, 2016, 12:41

Re: Review Request 45845: AMBARI-15694 : AMS returns truncated results when it exceeds the metrics service default result limit config

2016-04-06 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45845/#review127511 --- No unit test changes ? - Sid Wagle On April 6, 2016, 11:39

Re: Review Request 45744: AMBARI-15696: BulkCommand stop NodeManagers stops the wrong component

2016-04-06 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45744/#review127522 --- Ship it! Ship It! - Alejandro Fernandez On April 6, 2016,

Re: Review Request 45845: AMBARI-15694 : AMS returns truncated results when it exceeds the metrics service default result limit config

2016-04-06 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45845/#review127512 --- Ship it! Ship It! - Sid Wagle On April 6, 2016, 11:39 p.m.,

Re: Review Request 45841: Restart LLAP action should be shown only when Hive Interactive Server is present in the cluster

2016-04-06 Thread Srimanth Gunturi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45841/#review127508 --- Ship it! Ship It! - Srimanth Gunturi On April 6, 2016,

Re: Review Request 45837: Kafka jaas files are not generated on a Kafa upgrade if security.inter.broker.protocol is set to SASL_PLAINTEXT

2016-04-06 Thread Sriharsha Chintalapani
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45837/ --- (Updated April 7, 2016, 12:41 a.m.) Review request for Ambari, Alejandro

Review Request 45841: Restart LLAP action should be shown only when Hive Interactive Server is present in the cluster

2016-04-06 Thread Jaimin Jetly
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45841/ --- Review request for Ambari and Yusaku Sako. Bugs: AMBARI-15748

Re: Review Request 45832: Add ignore-bad-hosts flag to hawq activate standby custom action

2016-04-06 Thread Matt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45832/#review127562 --- Ship it!

Re: Review Request 45778: AMBARI-15577: Stack Featurize HIVE service

2016-04-06 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45778/#review127295 ---

Re: Review Request 45805: "E090 NullPointerException" when executing Hive queries with tez

2016-04-06 Thread Rohit Choudhary
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45805/#review127303 --- Ship it! Ship It! - Rohit Choudhary On April 6, 2016, 7:06

Re: Review Request 45374: Use ">>" instead of ">" to write ambari-metrics-collector.out

2016-04-06 Thread Dmytro Sen
> On Апрель 6, 2016, 9:05 д.п., Akira Ajisaka wrote: > > Can someone commit it? Committed to trunk. Thanks Akira. - Dmytro --- This is an automatically generated e-mail. To reply, visit:

Re: Review Request 45781: AMBARI-15722 [Ambari Web] move RedHat Satellite option out of experimental

2016-04-06 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45781/#review127306 --- Ship it! Ship It! - Alexandr Antonenko On April 5, 2016,

Re: Review Request 45538: Audit Log Code Cleanup & Safety

2016-04-06 Thread Daniel Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45538/ --- (Updated ápr. 6, 2016, 4:08 du) Review request for Ambari, Jonathan Hurley,

Re: Review Request 45811: Compatible Stacks not returning correctly

2016-04-06 Thread Nate Cole
> On April 6, 2016, 12:06 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/CompatibleRepositoryVersionResourceProvider.java, > > line 75 > > > > > > Does

Re: Review Request 45781: AMBARI-15722 [Ambari Web] move RedHat Satellite option out of experimental

2016-04-06 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45781/#review127321 --- Ship it! Is this hooked in anywhere? I don't see any API

Re: Review Request 45805: "E090 NullPointerException" when executing Hive queries with tez

2016-04-06 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45805/#review127316 --- Ship it! Ship It! - Pallav Kulshreshtha On April 6, 2016,

Re: Review Request 45538: Audit Log Code Cleanup & Safety

2016-04-06 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45538/#review127363 --- Ship it! Ship It! - Jonathan Hurley On April 6, 2016, 12:08

Re: Review Request 45781: AMBARI-15722 [Ambari Web] move RedHat Satellite option out of experimental

2016-04-06 Thread Zhe (Joe) Wang
> On April 6, 2016, 1:59 p.m., Nate Cole wrote: > > Is this hooked in anywhere? I don't see any API calls (but that's probably > > ok for this review). Any ambari-admin view changes? I thought the requirement for this issue is to expose the RedHat Satellite option, which used to be hidden

Re: Review Request 45791: AMBARI-15725: RBAC: Help page UI edits

2016-04-06 Thread Zhe (Joe) Wang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45791/#review127374 --- Ship it! Ship It! - Zhe (Joe) Wang On April 6, 2016, 1:20

Re: Review Request 45538: Audit Log Code Cleanup & Safety

2016-04-06 Thread Daniel Gergely
> On ápr. 4, 2016, 2:05 du, Jonathan Hurley wrote: > > ambari-server/src/main/java/org/apache/ambari/server/audit/request/eventcreator/AbstractBaseEventCreator.java, > > line 28 > > > > > > I'm actually a fan of

Re: Review Request 45374: Use ">>" instead of ">" to write ambari-metrics-collector.out

2016-04-06 Thread Akira Ajisaka
> On 4月 6, 2016, 9:05 a.m., Akira Ajisaka wrote: > > Can someone commit it? > > Dmytro Sen wrote: > Committed to trunk. > > Thanks Akira. Thank you, Dmytro and Aravindan! - Akira --- This is an automatically generated

Review Request 45815: Ambari DB schema upgrade fails with ORA-01408 error

2016-04-06 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45815/ --- Review request for Ambari and Myroslav Papirkovskyy. Bugs: AMBARI-15737

Re: Review Request 45815: Ambari DB schema upgrade fails with ORA-01408 error

2016-04-06 Thread Myroslav Papirkovskyy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45815/#review127348 --- Ship it! Ship It! - Myroslav Papirkovskyy On Квітень 6,