This is an automated email from the ASF dual-hosted git repository. stigahuang pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/impala.git
commit 22b59d27d0be25999eba4c839ea157c279939d76 Author: Michael Smith <[email protected]> AuthorDate: Thu Jul 18 14:28:41 2024 -0700 IMPALA-13243: Update Dropwizard Metrics to 4.2.x Updates Dropwizard Metrics components to the latest 4.2.x release, 4.2.26. We directly use metrics-core, and metrics-jvm/metrics-json are imported via Hive (via https://github.com/joshelser/dropwizard-hadoop-metrics2). Dropwizard Metrics manually tested with these versions on https://github.com/joshelser/dropwizard-hadoop-metrics2/pull/8. Change-Id: Ie9bec7a7c23194604430531bd83b25c5969e888e Reviewed-on: http://gerrit.cloudera.org:8080/21599 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- bin/impala-config.sh | 1 + fe/pom.xml | 1 - java/pom.xml | 19 +++++++++++++++++++ 3 files changed, 20 insertions(+), 1 deletion(-) diff --git a/bin/impala-config.sh b/bin/impala-config.sh index 91610fd8d..7c23c8006 100755 --- a/bin/impala-config.sh +++ b/bin/impala-config.sh @@ -297,6 +297,7 @@ export IMPALA_SPRINGFRAMEWORK_VERSION=5.3.37 export IMPALA_XMLSEC_VERSION=2.2.6 export IMPALA_OBS_VERSION=3.1.1-hw-42 export IMPALA_DBCP2_VERSION=2.9.0 +export IMPALA_DROPWIZARD_METRICS_VERSION=4.2.26 # When Impala is building docker images on Redhat-based distributions, # it is useful to be able to customize the base image. Some users will diff --git a/fe/pom.xml b/fe/pom.xml index 54b2094be..3a03b788b 100644 --- a/fe/pom.xml +++ b/fe/pom.xml @@ -392,7 +392,6 @@ under the License. <dependency> <groupId>io.dropwizard.metrics</groupId> <artifactId>metrics-core</artifactId> - <version>3.2.2</version> </dependency> <dependency> diff --git a/java/pom.xml b/java/pom.xml index 7d24fafc8..60966bf75 100644 --- a/java/pom.xml +++ b/java/pom.xml @@ -81,6 +81,7 @@ under the License. <json-smart.version>${env.IMPALA_JSON_SMART_VERSION}</json-smart.version> <commons-dbcp2.version>${env.IMPALA_DBCP2_VERSION}</commons-dbcp2.version> <log4j2.version>${env.IMPALA_LOG4J2_VERSION}</log4j2.version> + <dropwizard-metrics.version>${env.IMPALA_DROPWIZARD_METRICS_VERSION}</dropwizard-metrics.version> </properties> <repositories> @@ -388,6 +389,24 @@ under the License. <artifactId>log4j-core</artifactId> <version>${log4j2.version}</version> </dependency> + + <!-- Pin Dropwizard Metrics versions. We directly use metrics-core, and + metrics-jvm/metrics-json are imported via HMS. --> + <dependency> + <groupId>io.dropwizard.metrics</groupId> + <artifactId>metrics-core</artifactId> + <version>${dropwizard-metrics.version}</version> + </dependency> + <dependency> + <groupId>io.dropwizard.metrics</groupId> + <artifactId>metrics-jvm</artifactId> + <version>${dropwizard-metrics.version}</version> + </dependency> + <dependency> + <groupId>io.dropwizard.metrics</groupId> + <artifactId>metrics-json</artifactId> + <version>${dropwizard-metrics.version}</version> + </dependency> </dependencies> </dependencyManagement>
