[jira] [Commented] (HIVE-19942) Hive Notification: All events for indexes should have table name

["Vihang Karajgaonkar (JIRA)" <ji...@apache.org>]

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

Vihang Karajgaonkar commented on HIVE-19942:
--------------------------------------------

Thanks [~bharos92] for the patch. The change itself looks good to me. Can you please modify tests in {{TestDbNotificationListener}} to add assert that tbl_name is not null? 

> Hive Notification: All events for indexes should have table name
> ----------------------------------------------------------------
>
>                 Key: HIVE-19942
>                 URL: https://issues.apache.org/jira/browse/HIVE-19942
>             Project: Hive
>          Issue Type: Improvement
>          Components: Hive
>    Affects Versions: 2.3.2
>            Reporter: Bharathkrishna Guruvayoor Murali
>            Assignee: Bharathkrishna Guruvayoor Murali
>            Priority: Major
>         Attachments: HIVE-19942.1.patch
>
>
> All the events for indexes: Create Index, Alter Index, Drop Index have the TBL_NAME as null. The TBL_NAME should be populated with the table on which the index is created.
> This makes it easier to decide whether to process the event or not without needing to parse the json message (which is a slower process).



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)