hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Szehon Ho <sze...@cloudera.com>
Subject Re: Review Request 52084: HIVE-14775: Investigate IOException usage in Metrics APIs
Date Wed, 28 Sep 2016 17:48:10 GMT

-----------------------------------------------------------
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
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message