-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/39790/
-----------------------------------------------------------
Review request for Ambari, Dmytro Sen, Sumit Mohanty, and Sid Wagle.
Bugs: AMBARI-13642 and AMBARI-13643
https://issues.apache.org/jira/browse/AMBARI-13642
https://issues.apache.org/jira/browse/AMBARI-13643
Repository: ambari
Description
-------
AMBARI-13642 : Kafka fails to start unless AMS is installed
PROBLEM
When Kafka is installed on a cluster where AMS was never installed, the
installation fails.
BUG
The kafka.metrics.reporter property had a default value pointing to a Reporter
class provided by AMS.
FIX
Change default value of kafka.metrics.reporter to empty string. Change the
property to the appropriate AMS Kafka reporter class dynamically though stack
recommendations, if AMS is installed.
----------------------------------------------------------------------------
AMBARI-13643 : Warn if Xmn size is oversubscribed in AMS HBase master and
region server config
PROBLEM/BUG
While setting the AMS configuration, the user can set any unbounded value for
the Xmn settings. This might lead to undesirable outcomes from the JVM.
FIX
Provide validation of Xmn settings based on AMS mode.
If AMS is operating in embedded mode
regionserver_xmn ---- 12-20% of ( hbase_master_heapsize +
hbase_regionserver_heapsize)
If AMS is operating in distributed distributed mode
hbase_master_xmn ---- 12 - 20% of hbase_master_heapsize
regionserver_xmn ---- 12 - 20% of hbase_regionserver_heapsize
Diffs
-----
ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/package/scripts/params.py
3a348f8
ambari-server/src/main/resources/common-services/KAFKA/0.8.1.2.2/configuration/kafka-broker.xml
67a2318
ambari-server/src/main/resources/stacks/HDP/2.0.6/services/stack_advisor.py
5779211
ambari-server/src/main/resources/stacks/HDP/2.3/services/stack_advisor.py
b74298c
Diff: https://reviews.apache.org/r/39790/diff/
Testing
-------
Manual Testing done.
Relevant unit tests pass.
Thanks,
Aravindan Vijayan
