----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/74396/ -----------------------------------------------------------
Review request for ranger, Kishore Gopalakrishna, Madhan Neethiraj, Ramesh Mani, and Sailaja Polavarapu. Repository: ranger Description ------- RANGER-4047-Ranger KMS health metrics. The solution depends on "ranger-metrics" common module for both Json and Prometheus sink. Kms has added only KMS specific application metrics. Generally, one COUNT metric and corresponding elapsed time gauge metric for each REST end points. By default, metric collection is not thread-safe but the by adding following property in kms-site.xml it can be made thread-safe: Prop name: hadoop.kms.metric.collection.threadsafe Diffs ----- distro/src/main/assembly/kms.xml 4b4a2ac8e kms/pom.xml e97b993d7 kms/src/main/java/org/apache/hadoop/crypto/key/kms/server/KMS.java 12bcdc1aa kms/src/main/java/org/apache/hadoop/crypto/key/kms/server/KMSAuthenticationFilter.java 274bac910 kms/src/main/java/org/apache/hadoop/crypto/key/kms/server/KMSWebApp.java a32444cd1 kms/src/main/java/org/apache/hadoop/crypto/key/kms/server/MetricREST.java PRE-CREATION kms/src/main/java/org/apache/ranger/kms/metrics/KMSMetricWrapper.java PRE-CREATION kms/src/main/java/org/apache/ranger/kms/metrics/KMSMetrics.java PRE-CREATION kms/src/main/java/org/apache/ranger/kms/metrics/collector/KMSMetricsCollector.java PRE-CREATION kms/src/main/java/org/apache/ranger/kms/metrics/source/KMSMetricSource.java PRE-CREATION kms/src/main/resources/hadoop-metrics2.properties PRE-CREATION kms/src/test/java/org/apache/ranger/kms/metrics/TestKMSMetricsWrapper.java PRE-CREATION kms/src/test/resources/hadoop-metrics2.properties PRE-CREATION Diff: https://reviews.apache.org/r/74396/diff/1/ Testing ------- Build is successful but end-to-end functional testing is under-progress. Reviewers may start code review. Thanks, Vikas Kumar
