----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62487/#review186000 -----------------------------------------------------------
common/src/java/org/apache/hadoop/hive/common/metrics/metrics2/JsonFileMetricsReporter.java Line 53 (original), 68 (patched) <https://reviews.apache.org/r/62487/#comment262345> Its not clear looking at the code why we are using java.nio.file.Path instead of org.apache.hadoop.fs.Path. Can you please add a comment explain the same? - Vihang Karajgaonkar On Sept. 22, 2017, 4:01 p.m., Alexander Kolbasov wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/62487/ > ----------------------------------------------------------- > > (Updated Sept. 22, 2017, 4:01 p.m.) > > > Review request for hive, Aihua Xu, Sergio Pena, Sahil Takiar, and Vihang > Karajgaonkar. > > > 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 > > > Diffs > ----- > > > common/src/java/org/apache/hadoop/hive/common/metrics/metrics2/JsonFileMetricsReporter.java > c07517a634e35c936d6ea68e9a2874ac2c59929a > > common/src/test/org/apache/hadoop/hive/common/metrics/metrics2/TestCodahaleMetrics.java > 67f81d6c43a7904dc384cfcce6f948f1f802144c > > standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/metrics/JsonReporter.java > b804cdade07079955a65cb431fab078dcecd53b1 > > standalone-metastore/src/test/java/org/apache/hadoop/hive/metastore/metrics/TestMetrics.java > 259a4db43905c47d873776df96210a4e77d07076 > > > Diff: https://reviews.apache.org/r/62487/diff/3/ > > > Testing > ------- > > TestCodahaleMetrics tests that the json reporter functionality > > > Thanks, > > Alexander Kolbasov > >