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

Mohit Sabharwal commented on HIVE-17008:
----------------------------------------

{{ListenerEvent}} has the success flag, so listener implementation can choose 
to ignore failed events. But {{DbNotificationListener}} and 
{{HMSMetricsListener}} are ignoring the status flag.  The old 
{{NotificationListener}} though is looking at the flag and only logging 
successful events.

This does look like something we should make consistent, not just address the 
NPE. [~sushanth], [~thejas], [~akolb], do you have any thoughts ?

> HiveMetastore.drop_database can return NPE if database does not exist
> ---------------------------------------------------------------------
>
>                 Key: HIVE-17008
>                 URL: https://issues.apache.org/jira/browse/HIVE-17008
>             Project: Hive
>          Issue Type: Bug
>          Components: Metastore
>            Reporter: Dan Burkert
>            Assignee: Dan Burkert
>         Attachments: HIVE-17008.0.patch
>
>
> When dropping a non-existent database, the HMS will still fire registered 
> {{DROP_DATABASE}} event listeners.  This results in an NPE when the listeners 
> attempt to deref the {{null}} database parameter.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to