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

Vishal Khandelwal commented on HBASE-21047:
-------------------------------------------

Submitted the changes for branch-1.0 and after reviewing can submit the changes 
for master. For master code layout is little different and would need to update 
the changes accordingly.

All test expect below which was failing before the change

testAllowedClient(org.apache.hadoop.hbase.http.TestSpnegoHttpServer

[~apurtell]

> Object creation of StoreFileScanner thru constructor and close may leave 
> refCount to -1
> ---------------------------------------------------------------------------------------
>
>                 Key: HBASE-21047
>                 URL: https://issues.apache.org/jira/browse/HBASE-21047
>             Project: HBase
>          Issue Type: Bug
>            Reporter: Vishal Khandelwal
>            Assignee: Vishal Khandelwal
>            Priority: Major
>             Fix For: 1.5.0
>
>         Attachments: HBASE-21047.v1.patch
>
>
> During object creation  "*StoreFileScanner*", it does not increase the 
> refCount whereas while close it decrements the reader refCount. This will 
> cause refCount to -1 and isReadReference method was returning true 
> (refCount.get() != 0 This is causing store file not to be deleted. This may 
> also cause issue in situation when some thread is holding a scanner but it 
> may actually be closed due to above bug. Impact of this would be really high. 
> Attatching patch for the fix which makes sure if reference is held either 
> thru getScanner method or constructor, ref is always updated. Patch also 
> contains a test which validates the issue. 



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to