[ 
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)

Reply via email to