hbase-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Andrew Purtell (JIRA)" <j...@apache.org>
Subject [jira] [Updated] (HBASE-12575) Sanity check coprocessor classes and other classes from configuration are loadable
Date Sun, 07 Dec 2014 04:38:12 GMT

     [ https://issues.apache.org/jira/browse/HBASE-12575?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel

Andrew Purtell updated HBASE-12575:
    Attachment: HBASE-12575.patch

Here's a patch that isn't finished, just parking it here for comment. It adds checks for table
coprocessor specifications, using the CoprocessorClassLoader, when sanity checking the table
descriptor on the master. Although regionservers are the processes that will actually load
CP classes, we are already sanity checking compression on the master at table create time,
so have an assumption in place that the master environment mirrors what is available in the
regionserver environment.

If this is fine I'll proceed with class availability checks in HMaster#sanityCheckTableDescriptor
for store plugins.

> Sanity check coprocessor classes and other classes from configuration are loadable
> ----------------------------------------------------------------------------------
>                 Key: HBASE-12575
>                 URL: https://issues.apache.org/jira/browse/HBASE-12575
>             Project: HBase
>          Issue Type: Sub-task
>            Reporter: Enis Soztutar
>            Assignee: Andrew Purtell
>             Fix For: 1.0.0, 2.0.0, 0.98.9
>         Attachments: HBASE-12575.patch
> We load coprocessors and other classes from configuration. In case of a typo in the class
name (or deployment problem) a create table / alter table with wrong class name brings down
the whole cluster. 
> Master already does sanity check for compression and region split policy classes introduced
in HBASE-10591. We should extend that to check some other common cases as well. 

This message was sent by Atlassian JIRA

View raw message