-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62487/
-----------------------------------------------------------
(Updated Sept. 22, 2017, 7:25 a.m.)
Review request for hive, Aihua Xu, Sergio Pena, Sahil Takiar, and Vihang
Karajgaonkar.
Changes
-------
Fixed JsonReporter and its test
Bugs: HIVE-17563
https://issues.apache.org/jira/browse/HIVE-17563
Repository: hive-git
Description
-------
HIVE-17563 CodahaleMetrics.JsonFileReporter is not updating
hive.service.metrics.file.location
A few other changes included:
- Test clean up
- provide useful name for JSON reporter thread
Diffs (updated)
-----
common/src/java/org/apache/hadoop/hive/common/metrics/metrics2/JsonFileMetricsReporter.java
c07517a634
common/src/test/org/apache/hadoop/hive/common/metrics/metrics2/TestCodahaleMetrics.java
67f81d6c43
standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/metrics/JsonReporter.java
b804cdade0
standalone-metastore/src/test/java/org/apache/hadoop/hive/metastore/metrics/TestMetrics.java
259a4db439
Diff: https://reviews.apache.org/r/62487/diff/2/
Changes: https://reviews.apache.org/r/62487/diff/1-2/
Testing
-------
TestCodahaleMetrics tests that the json reporter functionality
Thanks,
Alexander Kolbasov
