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

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

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

     Result = SUCCESS
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

Reply via email to