hbase-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jerry He (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HBASE-13706) CoprocessorClassLoader should not exempt Hive classes
Date Wed, 05 Aug 2015 02:30:05 GMT

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

Jerry He commented on HBASE-13706:
----------------------------------

bq. If not going through a facade in org.apache.hadoop.hbase.* whatever objects the coprocessor
instantiates and interacts with will not have access to static shared state like UGI, the
metrics subsystem registry, the FileSystem instance cache, etc. Working with HDFS, metrics,
and security APIs would be "interesting"

Good points.  Maybe that is the right way?.  The current way is ambiguous and un-intended?
 Coprocessors should share with the host env only via clearly defined interfaces.
Yes, we can make the change on the master branch only.

> CoprocessorClassLoader should not exempt Hive classes
> -----------------------------------------------------
>
>                 Key: HBASE-13706
>                 URL: https://issues.apache.org/jira/browse/HBASE-13706
>             Project: HBase
>          Issue Type: Bug
>          Components: Coprocessors
>    Affects Versions: 2.0.0, 1.0.1, 1.1.0, 0.98.12
>            Reporter: Jerry He
>            Assignee: Jerry He
>            Priority: Minor
>             Fix For: 2.0.0, 0.98.14, 1.0.2, 1.1.3
>
>         Attachments: HBASE-13706.patch
>
>
> CoprocessorClassLoader is used to load classes from the coprocessor jar.
> Certain classes are exempt from being loaded by this ClassLoader, which means they will
be ignored in the coprocessor jar, but loaded from parent classpath instead.
> One problem is that we categorically exempt "org.apache.hadoop".
> But it happens that Hive packages start with "org.apache.hadoop".
> There is no reason to exclude hive classes from theCoprocessorClassLoader.
> HBase does not even include Hive jars.



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

Mime
View raw message