[
https://issues.apache.org/jira/browse/HBASE-9754?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13794855#comment-13794855
]
Lars Hofhansl commented on HBASE-9754:
--------------------------------------
Did a quick read through. Looks good. Need to look in more detail.
Using 0 as readPt when unknown is interesting.
Could we use the readPt to replace the hasMVCC stuff I just added in
HBASE-9751? I.e. pass MAX_LONG as readPt when we would have passed
hasMVCC=true, in which case we know we do not need to check the memstoreTs...?
I'd be in favor of committing HBASE-9753 first as that is simple, and
scrutinize this one a bit more.
> Consider eliminating threadlocals from MVCC code
> ------------------------------------------------
>
> Key: HBASE-9754
> URL: https://issues.apache.org/jira/browse/HBASE-9754
> Project: HBase
> Issue Type: Bug
> Reporter: Lars Hofhansl
> Assignee: Ted Yu
> Fix For: 0.98.0
>
> Attachments: 9754-rp-0.txt
>
>
> Brought up by [~vrodionov] and [[email protected]].
> Currently we use ThreadLocals to communicate the current readpoint between a
> RegionScanner and the Store\{File}Scanner's down the stack.
> Since ThreadLocals are not cheap we should consider whether it is possible to
> pass the readpoint through the call stack instead.
--
This message was sent by Atlassian JIRA
(v6.1#6144)
