----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52084/#review150736 -----------------------------------------------------------
Ship it! Ship It! - Szehon Ho On Sept. 28, 2016, 2:12 p.m., Barna Zsombor Klara wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/52084/ > ----------------------------------------------------------- > > (Updated Sept. 28, 2016, 2:12 p.m.) > > > Review request for hive, Gabor Szadovszky, Peter Vary, Sergio Pena, and > Szehon Ho. > > > Repository: hive-git > > > Description > ------- > > I have refactored the exception handling in the metrics API. The initial > objective was to remove the throws clause from the methods which did not > throw IOExceptions but were declaring it. Finally I continued to remove the > throws clause from the rest of the APIs since we were never able to do > anything else with the excpetions except log them out. New log messages were > introduced inside the metrics classes to capture exceptional conditions so we > can omit the try catch blocks when creating/updating metrics. > > > Diffs > ----- > > common/src/java/org/apache/hadoop/hive/common/metrics/LegacyMetrics.java > 9be9b50aa02ff88816eb92079eaff9afa3e1be90 > common/src/java/org/apache/hadoop/hive/common/metrics/MetricsMBean.java > 19946d9e93d55bbffb6d03cbc35e569849a86dd8 > common/src/java/org/apache/hadoop/hive/common/metrics/MetricsMBeanImpl.java > a973155f079c6124a6981b04123d9496dc5d3448 > common/src/java/org/apache/hadoop/hive/common/metrics/common/Metrics.java > 4297233ed12a7d9a2fa03ac3204e8335c0aed821 > > common/src/java/org/apache/hadoop/hive/common/metrics/metrics2/CodahaleMetrics.java > 4c433678bd62ea74b80babce9856681192deb25f > common/src/java/org/apache/hadoop/hive/ql/log/PerfLogger.java > 6a5d22f2bcc0a7efd869f7c0c0c8fad33ca35f74 > > common/src/test/org/apache/hadoop/hive/common/metrics/TestLegacyMetrics.java > a3fb04f1ab9be8be9f69c616eabeb534b2a2d560 > metastore/src/java/org/apache/hadoop/hive/metastore/HMSMetricsListener.java > 6830cf75dc163230cdabb4ed41d65c222c6ca54d > metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStore.java > 71175dffb6f425e971e2a16aa2233de2622919a3 > ql/src/java/org/apache/hadoop/hive/ql/Driver.java > 03c56e1d1838a5a239575a7fd8365dcc6787cce3 > ql/src/java/org/apache/hadoop/hive/ql/exec/mr/MapRedTask.java > f48d5110bcc0d37a37fbeb8abb50dc17876b04af > ql/src/java/org/apache/hadoop/hive/ql/exec/mr/MapredLocalTask.java > f81fc71d893dea44766eaf89c81825cd728babf7 > ql/src/java/org/apache/hadoop/hive/ql/exec/spark/SparkTask.java > 72c8bf792c2e5f2c14a7548ebfe7a0a7541e7115 > ql/src/java/org/apache/hadoop/hive/ql/exec/tez/TezTask.java > c51c92f1fc168633d00b69992e321d4be193f460 > service/src/java/org/apache/hive/service/cli/operation/Operation.java > 90fe76d00e6f833e18d183c290f13c23db9303a1 > > Diff: https://reviews.apache.org/r/52084/diff/ > > > Testing > ------- > > Tested basic metric gathering with both codahale and legacy metrics class. > Updated and ran unit tests in the common project. > > > Thanks, > > Barna Zsombor Klara > >
