dengzhhu653 commented on PR #4444:
URL: https://github.com/apache/hive/pull/4444#issuecomment-1767599913
Thanks @okumin for the great work! some ideas:
1. Can we just expose SessionHiveMetaStoreClient in the Hive class?
https://github.com/apache/hive/blob/master/ql/src/java/org/apache/hadoop/hive/ql/metadata/Hive.java#L5839
I think the `HiveMetaStoreClientWithLocalCache` can also benefit for
other data catalogs and it won't hurt the temp table by design.
2.
> I vote for users to specify an implementation of IMetaStoreClient via
HiveConf and use that in SessionHiveMetastoreClient instead of specifying a
factory class.
This makes sense to me, the SessionHiveMetastoreClient acts as a wrapper for
the IMetaStoreClient implementation.
3. IDataConnectorProvider in HMS can plugin different datasources nowadays,
example:
https://github.com/apache/hive/blob/master/standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/dataconnector/jdbc/PostgreSQLConnectorProvider.java,
which possible makes HMS a central meta repository across the organization, we
can add a Glue connector for HMS talking to Glue data catalog.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org
For additional commands, e-mail: gitbox-h...@hive.apache.org
