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

Enis Soztutar commented on HIVE-2720:
-------------------------------------

MetaStoreListener.onXXX() methods take an Event object to wrap the function 
call context, however HiveMetaHook functions does not. We can do either way, 
but the first one will introduce a lot of classes (50+) in case we do add 
corresponding methods. 

Since we want both preXXX() and onXXX() methods, we can do smt like:
preXXX() takes normal function args + context (containing back pointer to 
handler)
onXXX() takes normal function args + returned object + context (containing back 
pointer to handler + method status) 

Anyone care to comment ? 
                
> Merge MetaStoreListener and HiveMetaHook interfaces
> ---------------------------------------------------
>
>                 Key: HIVE-2720
>                 URL: https://issues.apache.org/jira/browse/HIVE-2720
>             Project: Hive
>          Issue Type: Sub-task
>          Components: JDBC, Metastore, ODBC, Security
>            Reporter: Enis Soztutar
>            Assignee: Enis Soztutar
>
> MetaStoreListener and HiveMetaHook both serve as a notification mechanism for 
> metastore-related events. The former is used by hcat and the latter is by the 
> hbase-storage handler, and invoked by the client. 
> I propose to merge these interfaces, and extend the MetaStoreListener, to add 
> most of the on- and pre- methods at the Thrift interface. This way, extending 
> metastore will be easier, and validation, storage-driver notification, and 
> enforcement can be delegated to individual listeners. Besides, more 
> functionality can be plugged-in by Hcat at this level. 

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to