[jira] [Created] (SENTRY-2518) Sentry should use HMS JSONFactory to parse notification events

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

Vihang Karajgaonkar created SENTRY-2518:
-------------------------------------------

             Summary: Sentry should use HMS JSONFactory to parse notification events
                 Key: SENTRY-2518
                 URL: https://issues.apache.org/jira/browse/SENTRY-2518
             Project: Sentry
          Issue Type: Improvement
            Reporter: Vihang Karajgaonkar


Sentry currently extends {{JSONMessageFactory}} class from Metastore and adds additional fields to the events to get the location. Metastore JSONMessageFactory has evolved over time and it includes the full-thrift objects in most event messages. Sentry should be able to fetch the required information by deserializing these thrift objects from the event message.

The advantage of doing this will be users can then switch to using the out-of-box notification message serializers and deserializers instead of custom events specific for Sentry. Thus any other metastore client which relies on the standard notification events can be used in addition to Sentry.

 

Note that this may mean either upgrading the metastore dependency to Hive-3 or backporting some of the hive patches to branch-2 of Hive. I did a quick check and it looks like we need to port the database object in {{JSONCreateDatabaseMessage}} and {{DropPartitionMessage}}. The other events already have all the necessary information required by Sentry.



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