[
https://issues.apache.org/jira/browse/IMPALA-10740?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Kishen Das updated IMPALA-10740:
--------------------------------
Description: Earlier we made MetastoreServiceHandler to extend
AbstractThriftHiveMetastore, which had the logic of throwing unsupported
operation exception for all HMS APIs. This only solves the build problem we had
on Impala, whenever a new HMS API is introduced. However, by default we need a
way to redirect the new HMS API to HMS itself, if we do not have to implement
the same in CatalogD. This new class DefaultThriftHiveMetastore in Hive does
exactly that. So, MetastoreServiceHandler should now extend
DefaultThriftHiveMetastore. (was: Earlier we made MetastoreServiceHandler to
extend AbstractThriftHiveMetastore, which had the logic of throwing unsupported
operation exception for all HMS APIs. This only solves the build problem we had
on Impala, whenever a new HMS API is introduced. However, by default we need a
way to redirect the new HMS API to HMS itself, if we do not have implement the
same in CatalogD. This new class DefaultThriftHiveMetastore in Hive does
exactly that. So, MetastoreServiceHandler should now extend
DefaultThriftHiveMetastore.)
> MetastoreServiceHandler should extend DefaultThriftHiveMetastore
> ----------------------------------------------------------------
>
> Key: IMPALA-10740
> URL: https://issues.apache.org/jira/browse/IMPALA-10740
> Project: IMPALA
> Issue Type: Improvement
> Components: Catalog
> Reporter: Kishen Das
> Assignee: Kishen Das
> Priority: Major
>
> Earlier we made MetastoreServiceHandler to extend
> AbstractThriftHiveMetastore, which had the logic of throwing unsupported
> operation exception for all HMS APIs. This only solves the build problem we
> had on Impala, whenever a new HMS API is introduced. However, by default we
> need a way to redirect the new HMS API to HMS itself, if we do not have to
> implement the same in CatalogD. This new class DefaultThriftHiveMetastore in
> Hive does exactly that. So, MetastoreServiceHandler should now extend
> DefaultThriftHiveMetastore.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
