----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62995/#review188416 -----------------------------------------------------------
common/src/java/org/apache/hadoop/hive/common/metrics/metrics2/JsonFileMetricsReporter.java Lines 119 (patched) <https://reviews.apache.org/r/62995/#comment265417> I think it is better to throw a RuntimeException here instead of catching it since there is nothing more that you can do if the metricsDir could not be created and it doesn't exist. - Vihang Karajgaonkar On Oct. 17, 2017, 12:35 a.m., Alexander Kolbasov wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/62995/ > ----------------------------------------------------------- > > (Updated Oct. 17, 2017, 12:35 a.m.) > > > Review request for hive, Aihua Xu, Andrew Sherman, Janaki Lahorani, Sergio > Pena, Sahil Takiar, and Vihang Karajgaonkar. > > > Bugs: HIVE-17806 > https://issues.apache.org/jira/browse/HIVE-17806 > > > Repository: hive-git > > > Description > ------- > > HIVE-17806 Create directory for metrics file if it doesn't exist > > > Diffs > ----- > > > common/src/java/org/apache/hadoop/hive/common/metrics/metrics2/JsonFileMetricsReporter.java > 96243cb74a154b9a639ffb080256c4b43bd35a4b > > common/src/test/org/apache/hadoop/hive/common/metrics/metrics2/TestCodahaleMetrics.java > 254af7d4310578e3883c0dffa64bed0f823696ea > > standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/metrics/JsonReporter.java > f71bb25463b56bc741f989454664397996b6a5cf > > > Diff: https://reviews.apache.org/r/62995/diff/2/ > > > Testing > ------- > > > Thanks, > > Alexander Kolbasov > >
