hive-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Dan Burkert (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HIVE-17402) Provide object location in the HMS notification messages
Date Tue, 10 Oct 2017 17:12:00 GMT

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

Dan Burkert commented on HIVE-17402:
------------------------------------

This patch is adding the location parameter to the [hcatalog notification message classes|https://github.com/apache/hive/tree/master/hcatalog/server-extensions/src/main/java/org/apache/hive/hcatalog/messaging/json],
which were deprecated as part of [HIVE-15180|https://issues.apache.org/jira/browse/HIVE-15180?focusedCommentId=15682862&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-15682862].
 The replacement [metastore notification message classes|https://github.com/apache/hive/tree/master/metastore/src/java/org/apache/hadoop/hive/metastore/messaging/json]
in many cases include a serialized full {{Table}} object, which internally includes the location
URI, so adding the location parameters should not be necessary if Sentry switches over to
use the new notification event classes.  This patch, as I understand it, will not have the
intended effect of adding the location URI to notification log entries, since the modified
classes are no longer used in the HMS.

> Provide object location in the HMS notification messages
> --------------------------------------------------------
>
>                 Key: HIVE-17402
>                 URL: https://issues.apache.org/jira/browse/HIVE-17402
>             Project: Hive
>          Issue Type: Improvement
>          Components: Hive
>    Affects Versions: 2.2.0
>            Reporter: Alexander Kolbasov
>            Assignee: Alexander Kolbasov
>         Attachments: HIVE-17402.01.patch
>
>
> While working on the Apache Sentry project that uses HMS notifications we noticed that
these notifications are using some useful data - e.g. location information for the objects.
To get around these, ApacheSentry implemented its own version of events (https://github.com/apache/sentry/tree/master/sentry-binding/sentry-binding-hive-follower/src/main/java/org/apache/sentry/binding/metastore/messaging/json).
> It seems to be a useful information for Hive as well, so why not add it directly into
the standard message factory?



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

Mime
View raw message