You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@hbase.apache.org by "Hudson (JIRA)" <ji...@apache.org> on 2012/11/05 02:36:33 UTC

[jira] [Commented] (HBASE-7073) OperationMetrics needs to cache the value of hbase.metrics.exposeOperationTimes

    [ https://issues.apache.org/jira/browse/HBASE-7073?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13490391#comment-13490391 ] 

Hudson commented on HBASE-7073:
-------------------------------

Integrated in HBase-0.94-security-on-Hadoop-23 #9 (See [https://builds.apache.org/job/HBase-0.94-security-on-Hadoop-23/9/])
    HBASE-7073 OperationMetrics needs to cache the value of hbase.metrics.exposeOperationTimes (Revision 1403865)

     Result = FAILURE
eclark : 
Files : 
* /hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/regionserver/metrics/OperationMetrics.java

                
> OperationMetrics needs to cache the value of hbase.metrics.exposeOperationTimes
> -------------------------------------------------------------------------------
>
>                 Key: HBASE-7073
>                 URL: https://issues.apache.org/jira/browse/HBASE-7073
>             Project: HBase
>          Issue Type: Improvement
>    Affects Versions: 0.94.2
>            Reporter: Jean-Daniel Cryans
>            Assignee: Elliott Clark
>            Priority: Minor
>             Fix For: 0.94.3
>
>         Attachments: HBASE-7073-0.patch
>
>
> Trying some increments on my local machine I was surprised to see this in my jstacks:
> {noformat}
>    java.lang.Thread.State: RUNNABLE
> 	at org.apache.hadoop.conf.Configuration.getProps(Configuration.java:1061)
> 	- locked <7c4a26430> (a org.apache.hadoop.conf.Configuration)
> 	at org.apache.hadoop.conf.Configuration.get(Configuration.java:416)
> 	at org.apache.hadoop.hbase.regionserver.CompoundConfiguration$1.get(CompoundConfiguration.java:94)
> 	at org.apache.hadoop.hbase.regionserver.CompoundConfiguration.get(CompoundConfiguration.java:186)
> 	at org.apache.hadoop.hbase.regionserver.CompoundConfiguration.getBoolean(CompoundConfiguration.java:318)
> 	at org.apache.hadoop.hbase.regionserver.metrics.OperationMetrics.doSafeIncTimeVarying(OperationMetrics.java:217)
> 	at org.apache.hadoop.hbase.regionserver.metrics.OperationMetrics.doUpdateTimeVarying(OperationMetrics.java:212)
> 	at org.apache.hadoop.hbase.regionserver.metrics.OperationMetrics.updateIncrementMetrics(OperationMetrics.java:133)
> 	at org.apache.hadoop.hbase.regionserver.HRegion.increment(HRegion.java:4817)
> {noformat}
> It's a pretty horrible lookup that's inline with everything else in that class and there's no reason why it shouldn't be a final boolean.
> Assigning this to the master of metrics since he asked for it.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira