[
https://issues.apache.org/jira/browse/HBASE-9889?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13815590#comment-13815590
]
Amitanand Aiyer commented on HBASE-9889:
----------------------------------------
I think we can move it to the end of the constructor.
Just need to make sure that we grab the read points before we open the scanners
(have seen get latency go up .. if we move the entier synchronized () .. block
to the end).
for the removal .. i think it is okay. scannerReadPoints is supposed to be a
concurrentHashMap.
> Make sure we clean up scannerReadPoints upon any exceptions
> -----------------------------------------------------------
>
> Key: HBASE-9889
> URL: https://issues.apache.org/jira/browse/HBASE-9889
> Project: HBase
> Issue Type: Sub-task
> Affects Versions: 0.89-fb, 0.94.12, 0.96.0
> Reporter: Amitanand Aiyer
> Assignee: Amitanand Aiyer
> Priority: Minor
> Fix For: 0.96.1
>
> Attachments: hbase-9889.diff
>
>
> If there is an exception in the creation of RegionScanner (for example,
> exception while opening store files) the scanner Read points is not cleaned
> up.
> Having an unused old entry in the scannerReadPoints means that flushes and
> compactions cannot garbage-collect older versions.
--
This message was sent by Atlassian JIRA
(v6.1#6144)
