You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@hbase.apache.org by "Himanshu Vashishtha (JIRA)" <ji...@apache.org> on 2011/05/01 04:03:03 UTC

[jira] [Created] (HBASE-3838) RegionCoprocesorHost.preWALRestore throws npe in case there is no RegionObserver registered.

RegionCoprocesorHost.preWALRestore throws npe in case there is no RegionObserver registered.
--------------------------------------------------------------------------------------------

                 Key: HBASE-3838
                 URL: https://issues.apache.org/jira/browse/HBASE-3838
             Project: HBase
          Issue Type: Bug
            Reporter: Himanshu Vashishtha
            Priority: Minor


It seems the check to bypass the Observers chain is at wrong place in case of pre/post WALRestore. It should be inside the "if statement" that checks whether the CP is instance of RegionObserver or not.


--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

[jira] [Updated] (HBASE-3838) RegionCoprocesorHost.preWALRestore throws npe in case there is no RegionObserver registered.

Posted by "Himanshu Vashishtha (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HBASE-3838?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Himanshu Vashishtha updated HBASE-3838:
---------------------------------------

    Attachment: patch.txt

correcting the byPass and shouldComplete check in RegionCoprocessorHost.

> RegionCoprocesorHost.preWALRestore throws npe in case there is no RegionObserver registered.
> --------------------------------------------------------------------------------------------
>
>                 Key: HBASE-3838
>                 URL: https://issues.apache.org/jira/browse/HBASE-3838
>             Project: HBase
>          Issue Type: Bug
>            Reporter: Himanshu Vashishtha
>            Priority: Minor
>         Attachments: patch.txt
>
>
> It seems the check to bypass the Observers chain is at wrong place in case of pre/post WALRestore. It should be inside the "if statement" that checks whether the CP is instance of RegionObserver or not.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

[jira] [Resolved] (HBASE-3838) RegionCoprocesorHost.preWALRestore throws npe in case there is no RegionObserver registered.

Posted by "stack (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HBASE-3838?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

stack resolved HBASE-3838.
--------------------------

       Resolution: Fixed
    Fix Version/s: 0.92.0
         Assignee: Himanshu Vashishtha
     Hadoop Flags: [Reviewed]

Committed to TRUNK.  Thank you for the patch Himanshu.

> RegionCoprocesorHost.preWALRestore throws npe in case there is no RegionObserver registered.
> --------------------------------------------------------------------------------------------
>
>                 Key: HBASE-3838
>                 URL: https://issues.apache.org/jira/browse/HBASE-3838
>             Project: HBase
>          Issue Type: Bug
>            Reporter: Himanshu Vashishtha
>            Assignee: Himanshu Vashishtha
>            Priority: Minor
>             Fix For: 0.92.0
>
>         Attachments: patch.txt
>
>
> It seems the check to bypass the Observers chain is at wrong place in case of pre/post WALRestore. It should be inside the "if statement" that checks whether the CP is instance of RegionObserver or not.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

[jira] [Commented] (HBASE-3838) RegionCoprocesorHost.preWALRestore throws npe in case there is no RegionObserver registered.

Posted by "Hudson (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HBASE-3838?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13030202#comment-13030202 ] 

Hudson commented on HBASE-3838:
-------------------------------

Integrated in HBase-TRUNK #1909 (See [https://builds.apache.org/hudson/job/HBase-TRUNK/1909/])
    

> RegionCoprocesorHost.preWALRestore throws npe in case there is no RegionObserver registered.
> --------------------------------------------------------------------------------------------
>
>                 Key: HBASE-3838
>                 URL: https://issues.apache.org/jira/browse/HBASE-3838
>             Project: HBase
>          Issue Type: Bug
>            Reporter: Himanshu Vashishtha
>            Assignee: Himanshu Vashishtha
>            Priority: Minor
>             Fix For: 0.92.0
>
>         Attachments: patch.txt
>
>
> It seems the check to bypass the Observers chain is at wrong place in case of pre/post WALRestore. It should be inside the "if statement" that checks whether the CP is instance of RegionObserver or not.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira