hbase-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Gregory Chanan (JIRA)" <j...@apache.org>
Subject [jira] [Updated] (HBASE-6333) MemStoreScanner {kvsetAtCreation,snapshotAtCreation} should be private and not volatile
Date Thu, 05 Jul 2012 23:58:33 GMT

     [ https://issues.apache.org/jira/browse/HBASE-6333?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel

Gregory Chanan updated HBASE-6333:

    Attachment: HBASE-6333.patch
> MemStoreScanner {kvsetAtCreation,snapshotAtCreation} should be private and not volatile
> ---------------------------------------------------------------------------------------
>                 Key: HBASE-6333
>                 URL: https://issues.apache.org/jira/browse/HBASE-6333
>             Project: HBase
>          Issue Type: Improvement
>    Affects Versions: 0.96.0
>            Reporter: Gregory Chanan
>            Assignee: Gregory Chanan
>            Priority: Minor
>         Attachments: HBASE-6333.patch
> I was looking over this code and was confused why these variables were volatile, as they
don't need to be [the constructor, which is the only place the variables are set, is already
protected by a lock].
> I doubt this makes much performance difference (I haven't measured); this is more of
a documentation difference to say nothing special is going on here :).
> https://issues.apache.org/jira/browse/HBASE-4485?focusedCommentId=13132733&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-13132733
also talks about this, but it doesn't seem to have been implemented.

This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira


View raw message