You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@ambari.apache.org by Jungtaek Lim <ka...@gmail.com> on 2017/12/25 04:36:10 UTC
Review Request 64836: AMBARI-22696 Whitelist execute latency from
Storm Ambari metrics (trunk)
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64836/
-----------------------------------------------------------
Review request for Ambari, Arun Mahadevan and Aravindan Vijayan.
Bugs: AMBARI-22696
https://issues.apache.org/jira/browse/AMBARI-22696
Repository: ambari
Description
-------
We have an use case which leverages execute latency along with process latency. Given that we only register process latency as whitelist, we would need to add execute latency as whitelist as well.
Diffs
-----
ambari-server/src/main/resources/common-services/STORM/1.0.1.3.0/configuration/storm-site.xml 6b97fb6e41
ambari-server/src/main/resources/common-services/STORM/1.0.1.3.0/service_advisor.py 1d6bbe0f9f
ambari-server/src/main/resources/common-services/STORM/1.0.1/configuration/storm-site.xml 558beaf8d9
ambari-server/src/main/resources/stacks/HDP/2.5/services/stack_advisor.py 6f409cdfd1
ambari-server/src/test/python/stacks/2.5/common/test_stack_advisor.py cc6bd9dca4
Diff: https://reviews.apache.org/r/64836/diff/1/
Testing
-------
Manually applied the change to configuration of test cluster, and confirmed it works.
It doesn't mean I have tested it with upgrading, so please consider verifying it works well with various upgrade cases.
Thanks,
Jungtaek Lim
Re: Review Request 64836: AMBARI-22696 Whitelist execute latency from
Storm Ambari metrics (trunk)
Posted by Aravindan Vijayan <av...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64836/#review194562
-----------------------------------------------------------
Ship it!
Ship It!
- Aravindan Vijayan
On Dec. 25, 2017, 4:36 a.m., Jungtaek Lim wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/64836/
> -----------------------------------------------------------
>
> (Updated Dec. 25, 2017, 4:36 a.m.)
>
>
> Review request for Ambari, Arun Mahadevan and Aravindan Vijayan.
>
>
> Bugs: AMBARI-22696
> https://issues.apache.org/jira/browse/AMBARI-22696
>
>
> Repository: ambari
>
>
> Description
> -------
>
> We have an use case which leverages execute latency along with process latency. Given that we only register process latency as whitelist, we would need to add execute latency as whitelist as well.
>
>
> Diffs
> -----
>
> ambari-server/src/main/resources/common-services/STORM/1.0.1.3.0/configuration/storm-site.xml 6b97fb6e41
> ambari-server/src/main/resources/common-services/STORM/1.0.1.3.0/service_advisor.py 1d6bbe0f9f
> ambari-server/src/main/resources/common-services/STORM/1.0.1/configuration/storm-site.xml 558beaf8d9
> ambari-server/src/main/resources/stacks/HDP/2.5/services/stack_advisor.py 6f409cdfd1
> ambari-server/src/test/python/stacks/2.5/common/test_stack_advisor.py cc6bd9dca4
>
>
> Diff: https://reviews.apache.org/r/64836/diff/1/
>
>
> Testing
> -------
>
> Manually applied the change to configuration of test cluster, and confirmed it works.
>
> It doesn't mean I have tested it with upgrading, so please consider verifying it works well with various upgrade cases.
>
>
> Thanks,
>
> Jungtaek Lim
>
>
Re: Review Request 64836: AMBARI-22696 Whitelist execute latency from
Storm Ambari metrics (trunk)
Posted by Arun Mahadevan <ar...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64836/#review194496
-----------------------------------------------------------
Ship it!
Ship It!
- Arun Mahadevan
On Dec. 25, 2017, 4:36 a.m., Jungtaek Lim wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/64836/
> -----------------------------------------------------------
>
> (Updated Dec. 25, 2017, 4:36 a.m.)
>
>
> Review request for Ambari, Arun Mahadevan and Aravindan Vijayan.
>
>
> Bugs: AMBARI-22696
> https://issues.apache.org/jira/browse/AMBARI-22696
>
>
> Repository: ambari
>
>
> Description
> -------
>
> We have an use case which leverages execute latency along with process latency. Given that we only register process latency as whitelist, we would need to add execute latency as whitelist as well.
>
>
> Diffs
> -----
>
> ambari-server/src/main/resources/common-services/STORM/1.0.1.3.0/configuration/storm-site.xml 6b97fb6e41
> ambari-server/src/main/resources/common-services/STORM/1.0.1.3.0/service_advisor.py 1d6bbe0f9f
> ambari-server/src/main/resources/common-services/STORM/1.0.1/configuration/storm-site.xml 558beaf8d9
> ambari-server/src/main/resources/stacks/HDP/2.5/services/stack_advisor.py 6f409cdfd1
> ambari-server/src/test/python/stacks/2.5/common/test_stack_advisor.py cc6bd9dca4
>
>
> Diff: https://reviews.apache.org/r/64836/diff/1/
>
>
> Testing
> -------
>
> Manually applied the change to configuration of test cluster, and confirmed it works.
>
> It doesn't mean I have tested it with upgrading, so please consider verifying it works well with various upgrade cases.
>
>
> Thanks,
>
> Jungtaek Lim
>
>