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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45778/#review127295 --- ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45805/ --- Review request for Ambari, DIPAYAN BHOWMICK, Nitiraj Rathore, Pallav Kulshreshth

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

--- 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 a

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45374/#review127304 --- Can someone commit it? - Akira Ajisaka On 3月 30, 2016, 2:02 a.m

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45805/#review127305 --- Ship it! Ship It! - Nitiraj Rathore On April 6, 2016, 7:06 a

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

--- 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, 10:

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45809/ --- Review request for Ambari and Vitalyi Brodetskyi. Bugs: AMBARI-15729 https:

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

--- 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 Brode

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

--- 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, 1

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

> 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: https://reviews.apache.org/r/45374/#review127304

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

--- 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, 7:

Re: Review Request 45676: AMBARI-15678: YARN service_check doesn't fail when application status is not reasonable

> On April 4, 2016, 6:37 p.m., Vitalyi Brodetskyi wrote: > > Ship It! Thank you. Could you commit this ? - Masahiro --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45676/#review126892 ---

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45810/ --- Review request for Ambari, Alejandro Fernandez, Rohit Choudhary, Renjith Kamath,

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

--- 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 calls

Review Request 45811: Compatible Stacks not returning correctly

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45811/ --- Review request for Ambari, Alejandro Fernandez, Jonathan Hurley, and Jayush Luni

Re: Review Request 45811: Compatible Stacks not returning correctly

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45811/#review127328 --- Ship it! ambari-server/src/main/java/org/apache/ambari/server/

Re: Review Request 44210: AMBARI-15226 - The stack advisor should be pushed down to the services

> On March 2, 2016, 9:37 p.m., jun aoki wrote: > > ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HBASE/service_advisor_HDP206HBASE.py, > > line 1 > > > > > > Hi Tim, nice effort to decouple stack_advisor!

Re: Review Request 45811: Compatible Stacks not returning correctly

> On April 6, 2016, 10:47 a.m., Jonathan Hurley wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/CompatibleRepositoryVersionResourceProvider.java, > > line 250 > > > > > > Does 2 r

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

> 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 float

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45538/ --- (Updated ápr. 6, 2016, 3:19 du) Review request for Ambari, Jonathan Hurley, Nat

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

> On ápr. 4, 2016, 1:30 du, Sebastian Toader wrote: > > ambari-server/src/main/java/org/apache/ambari/server/api/services/BaseService.java, > > line 127 > > > > > > If requestBodySet is not empty than the last 'resu

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

> 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 comp

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

> 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 e-ma

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

--- 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 htt

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

--- 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, 2016

Re: Review Request 45811: Compatible Stacks not returning correctly

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45811/#review127353 --- ambari-server/src/main/java/org/apache/ambari/server/controller/i

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

--- 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, Nat

Re: Review Request 45811: Compatible Stacks not returning correctly

> 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 t

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

--- 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

> 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 wi

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

--- 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 a.

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45810/#review127376 --- Ship it! Ship It! - Jayush Luniya On April 6, 2016, 1:10 p.m

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

--- 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, 1:

Review Request 45824: Failed to install oozie client

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45824/ --- Review request for Ambari and Sumit Mohanty. Bugs: AMBARI-15739 https://iss

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

--- 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,

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

--- 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 45826: Descriptions for HAWQ sysctl params mention incorrect default values

--- 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 p.m.,

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

--- 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., Alexa

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

--- 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 p.m.

Re: Review Request 45824: Failed to install oozie client

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

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

--- 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 Fern

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

--- 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 De

Re: Review Request 45824: Failed to install oozie client

--- 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 p.m

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

> 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 45811: Compatible Stacks not returning correctly

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45811/#review127409 --- Ship it! Ship It! - Jayush Luniya On April 6, 2016, 2:03 p.m

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

> 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 float,

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

--- 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: AMBAR

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

> 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 float

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

--- 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 45833: Webhcat Server failed to stop while stopping all the services

--- 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 45834: Create Alert For Reporting Potential Issues With Slow REST Responses

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45834/ --- Review request for Ambari, Alejandro Fernandez, Dmitro Lisnichenko, and Nate Col

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

> 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 comm

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

--- 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 Fernand

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

--- 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 p.m

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

--- 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 Fernand

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

--- 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 Fernand

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

> 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 durin

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

--- 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 p.m

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45835/#review127461 --- ambari-server/src/main/resources/common-services/KAFKA/0.8.1.2.2/

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

--- 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, 9:

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45837/#review127481 --- ambari-server/src/main/resources/common-services/KAFKA/0.8.1.2.2/

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

> 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 ty

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

> 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 ty

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

--- 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 https://issue

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

--- 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 p.m.

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

--- 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, 11:25

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

--- 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: AMBA

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45845/ --- (Updated April 6, 2016, 11:39 p.m.) Review request for Ambari, Dmytro Sen, Sumi

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

--- 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 p.m.

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

--- 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 45744: AMBARI-15696: BulkCommand stop NodeManagers stops the wrong component

--- 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, 6:

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45778/#review127523 --- Fix it, then Ship it! ambari-server/src/main/resources/common-

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

--- 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 Fernan

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

--- 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 a.

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

--- 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 Hurley,

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45832/#review127562 --- Ship it! ambari-server/src/main/resources/common-services/HAWQ