hbase-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Hudson (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HBASE-7073) OperationMetrics needs to cache the value of hbase.metrics.exposeOperationTimes
Date Wed, 31 Oct 2012 04:11:12 GMT

    [ 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

Mime
View raw message