> On Sept. 22, 2017, 7:48 p.m., Sahil Takiar wrote: > > common/src/java/org/apache/hadoop/hive/common/metrics/metrics2/JsonFileMetricsReporter.java > > Line 75 (original), 116 (patched) > > <https://reviews.apache.org/r/62487/diff/4/?file=1832899#file1832899line129> > > > > I think the old implementation called `shutdown()` instead of > > `shutdownNow()`. It may be best to call `shutdown()` first, call > > `awaitTermination(30s)`, and then call `shutdownNow()`.
Reverted to the old code (calling shutdown()) > On Sept. 22, 2017, 7:48 p.m., Sahil Takiar wrote: > > standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/metrics/JsonReporter.java > > Lines 107 (patched) > > <https://reviews.apache.org/r/62487/diff/4/?file=1832901#file1832901line110> > > > > Might want to change this to something more informative, like `logging > > metrics with frequency: ` Removed. - Alexander ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62487/#review186011 ----------------------------------------------------------- On Sept. 22, 2017, 6:14 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, 6:14 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/4/ > > > Testing > ------- > > TestCodahaleMetrics tests that the json reporter functionality > > > Thanks, > > Alexander Kolbasov > >
