[
https://issues.apache.org/jira/browse/HBASE-12575?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14238765#comment-14238765
]
Enis Soztutar commented on HBASE-12575:
---------------------------------------
This looks good.
nit: testTableCoprocessorAttrs can be named checkTableCoprocessorAttrs().
What about other classes pluggable? We have a lot of those, it might take some
time to cover everything though. I would be ok to commit this as it is.
> Sanity check table coprocessor classes 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, HBASE-12575.patch, HBASE-12575.patch,
> HBASE-12575.patch, 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
(v6.3.4#6332)
