This is an automated email from the ASF dual-hosted git repository. wuzhiguo pushed a commit to branch trunk in repository https://gitbox.apache.org/repos/asf/ambari.git
The following commit(s) were added to refs/heads/trunk by this push: new 0a672bfdc0 AMBARI-25852: Change Kafka metrics_reporters param setup logic (#3640) 0a672bfdc0 is described below commit 0a672bfdc061bce566b33e48601fdfd0ffdaed00 Author: Zhiguo Wu <wuzhi...@apache.org> AuthorDate: Thu Feb 16 15:48:40 2023 +0800 AMBARI-25852: Change Kafka metrics_reporters param setup logic (#3640) --- .../stacks/BIGTOP/3.2.0/services/KAFKA/configuration/kafka-broker.xml | 2 +- .../stacks/BIGTOP/3.2.0/services/KAFKA/package/scripts/params.py | 3 +++ .../resources/stacks/BIGTOP/3.2.0/services/KAFKA/service_advisor.py | 4 ---- 3 files changed, 4 insertions(+), 5 deletions(-) diff --git a/ambari-server/src/main/resources/stacks/BIGTOP/3.2.0/services/KAFKA/configuration/kafka-broker.xml b/ambari-server/src/main/resources/stacks/BIGTOP/3.2.0/services/KAFKA/configuration/kafka-broker.xml index 59458ad214..d886d7218d 100644 --- a/ambari-server/src/main/resources/stacks/BIGTOP/3.2.0/services/KAFKA/configuration/kafka-broker.xml +++ b/ambari-server/src/main/resources/stacks/BIGTOP/3.2.0/services/KAFKA/configuration/kafka-broker.xml @@ -305,7 +305,7 @@ </property> <property> <name>kafka.metrics.reporters</name> - <value /> + <value>{{metrics_reporters}}</value> <description> kafka ganglia metrics reporter and kafka timeline metrics reporter </description> diff --git a/ambari-server/src/main/resources/stacks/BIGTOP/3.2.0/services/KAFKA/package/scripts/params.py b/ambari-server/src/main/resources/stacks/BIGTOP/3.2.0/services/KAFKA/package/scripts/params.py index f68442ef6e..2ee2d572ca 100644 --- a/ambari-server/src/main/resources/stacks/BIGTOP/3.2.0/services/KAFKA/package/scripts/params.py +++ b/ambari-server/src/main/resources/stacks/BIGTOP/3.2.0/services/KAFKA/package/scripts/params.py @@ -153,6 +153,9 @@ if has_metric_collector: else: metric_collector_protocol = 'http' + # If AMS is part of Services, use the KafkaTimelineMetricsReporter for metric reporting. Default is ''. + metrics_reporters = "org.apache.hadoop.metrics2.sink.kafka.KafkaTimelineMetricsReporter" + host_in_memory_aggregation = str(default("/configurations/ams-site/timeline.metrics.host.inmemory.aggregation", True)).lower() host_in_memory_aggregation_port = default("/configurations/ams-site/timeline.metrics.host.inmemory.aggregation.port", 61888) is_aggregation_https_enabled = False diff --git a/ambari-server/src/main/resources/stacks/BIGTOP/3.2.0/services/KAFKA/service_advisor.py b/ambari-server/src/main/resources/stacks/BIGTOP/3.2.0/services/KAFKA/service_advisor.py index 33df7ac2ca..43e426b9b7 100644 --- a/ambari-server/src/main/resources/stacks/BIGTOP/3.2.0/services/KAFKA/service_advisor.py +++ b/ambari-server/src/main/resources/stacks/BIGTOP/3.2.0/services/KAFKA/service_advisor.py @@ -263,10 +263,6 @@ class KafkaRecommender(service_advisor.ServiceAdvisor): else: putKafkaBrokerAttributes('authorizer.class.name', 'delete', 'true') - #If AMS is part of Services, use the KafkaTimelineMetricsReporter for metric reporting. Default is ''. - if "AMBARI_METRICS" in servicesList: - putKafkaBrokerProperty('kafka.metrics.reporters', 'org.apache.hadoop.metrics2.sink.kafka.KafkaTimelineMetricsReporter') - if ranger_plugin_enabled: kafkaLog4jRangerLines = [{ "name": "log4j.appender.rangerAppender", --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@ambari.apache.org For additional commands, e-mail: commits-h...@ambari.apache.org