[
https://issues.apache.org/jira/browse/HIVE-24413?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Stamatis Zampetakis updated HIVE-24413:
---------------------------------------
Fix Version/s: (was: All Versions)
I cleared the fixVersion field since this ticket is still open. Please review
this ticket and if the fix is already committed to a specific version please
set the version accordingly and mark the ticket as RESOLVED.
According to the [JIRA
guidelines|https://cwiki.apache.org/confluence/display/Hive/HowToContribute]
the fixVersion should be set only when the issue is resolved/closed.
> MetaStorePreEventListener is incomplete
> ---------------------------------------
>
> Key: HIVE-24413
> URL: https://issues.apache.org/jira/browse/HIVE-24413
> Project: Hive
> Issue Type: Improvement
> Components: Metastore
> Affects Versions: 3.1.2
> Reporter: Alexey Vishnevskiy
> Priority: Major
>
> According to Hive documentation, MetaStorePreEventListener should be used to
> perform authorization and/or additional actions for every metastore call.
> MetaStorePreEventListener is implemented and works for some calls, but does
> not work for others. For example, It does work for calls: getTable,
> createDatabase, createTable, alterTable, alterPartition, etc... but does not
> get called for alterDatabase, getFields, getDatabases, getTableStatistics,
> etc...
> Proper implementation of MetaStorePreEventListener would require a lot of
> changes in different classes so it seems that it is better to add a similar
> listener in the descendant of org.apache.thrift.ProcessFunction and change
> ThriftHiveMetastore generated by thrift compiler to descend all
> auto-generated classes from it, not from the ProcessFunction directly. This
> applies to all versions of HiveMetastore with MetaStorePreEventListener
> implemented.
> I would like to contribute code for this functionality for the trunk - please
> let me know your suggestions and thoughts.
> Thanks!
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
