----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/352/ -----------------------------------------------------------
(Updated 2011-02-02 10:23:23.214832) Review request for hive. Changes ------- Thanks for the review! I've updated the diff: - Added open/reopen semantics to MetricsScope, and made Metrics cache the older scopes. - added requested stack trace - edited for ){ , }else{ , spaces around operators, spaces between arguments, and 100 char limit Summary ------- HIVE-1818 : jmx instrumentation for HiveMetaStore call frequency and duration This addresses bug HIVE-1818. https://issues.apache.org/jira/browse/HIVE-1818 Diffs (updated) ----- http://svn.apache.org/repos/asf/hive/trunk/common/src/java/org/apache/hadoop/hive/common/metrics/Metrics.java PRE-CREATION http://svn.apache.org/repos/asf/hive/trunk/common/src/java/org/apache/hadoop/hive/common/metrics/MetricsMBean.java PRE-CREATION http://svn.apache.org/repos/asf/hive/trunk/common/src/java/org/apache/hadoop/hive/common/metrics/MetricsMBeanImpl.java PRE-CREATION http://svn.apache.org/repos/asf/hive/trunk/metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStore.java 1066213 http://svn.apache.org/repos/asf/hive/trunk/metastore/src/test/org/apache/hadoop/hive/metastore/TestHiveMetaStore.java 1066213 Diff: https://reviews.apache.org/r/352/diff Testing ------- Thanks, Sushanth