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

Sushanth Sowmyan commented on HIVE-9271:
----------------------------------------

Mostly looks good to me, and has my tentative +1.

One minor thing : I wondered if fire_notification_event shouldn't return a long 
instead of a void, thus returning the notification id of the fired event - but 
that's a strict no-no from a perspective of what a MetaStoreEventListener is 
supposed to be, since a NotificationListener is only one type of listener. That 
got me thinking - should this method be called fire_event rather than 
fire_notification_event?



> Add ability for client to request metastore to fire an event
> ------------------------------------------------------------
>
>                 Key: HIVE-9271
>                 URL: https://issues.apache.org/jira/browse/HIVE-9271
>             Project: Hive
>          Issue Type: New Feature
>          Components: Metastore
>            Reporter: Alan Gates
>            Assignee: Alan Gates
>             Fix For: 0.15.0
>
>         Attachments: HIVE-9271.patch
>
>
> Currently all events in Hive are fired by the metastore.  However, there are 
> events that only the client fully understands, such as DML operations.  There 
> should be a way for the client to request the metastore to fire a particular 
> event.



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

Reply via email to