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

Ted Yu commented on HBASE-16245:
--------------------------------

The change in conf/hbase-site.xml seems unrelated to the fix.
{code}
5916            if (storeHeap == null) {
5917            // scanner is closed
{code}
{code}
5920                    // Let's see what we have in the storeHeap.
5918            Cell current = this.storeHeap.peek();   5921            Cell 
current = this.storeHeap.peek();
{code}
Please properly align the comments.

> RegionScannerImpl#nextInternal() may produce null pointer exception if 
> scanner is closed in between
> ---------------------------------------------------------------------------------------------------
>
>                 Key: HBASE-16245
>                 URL: https://issues.apache.org/jira/browse/HBASE-16245
>             Project: HBase
>          Issue Type: Bug
>            Reporter: Ankit Singhal
>         Attachments: HBASE-16245.patch
>
>
> This issue is similar to HBASE-12274, FYI [~te...@apache.org]
> If I follow the discussion on HBASE-12274 and synchronization is not the 
> option.
> then the attached patch handles null pointer exception in a same way as 
> HBASE-12274



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to