FANNG

> @SteNicholas @FANNG1 can you please move forward this PR? plan to review this PR today or tomorrow.

> @FANNG1, IMO, splitting this pull request have certain works. Could this pull request take a review firstly? @SteNicholas, the overall architecture LGTM, do you like to split to smaller...

Iceberg catalog use a specific parameter `table_type` to check whether it's Iceberg table ```java List tableNames = clients.run(client -> client.getAllTables(database)); List tableIdentifiers; if (listAllTables) { tableIdentifiers = tableNames.stream() .map(t ->...

@danhuawang , it's mainly caused by using `spark-sql` and not setting `spark.sql.hive.metastore.jars` explicitly. I create a issue in https://github.com/apache/kyuubi/issues/6362, and will create a new PR to allow setting `spark.sql.hive.metastore.jars` to...

@danhuawang , you could download hive jars to your machine and set corresponding catalog properties like below to use spark-sql ``` { "name": "hive", "type": "RELATIONAL", "comment": "comment", "provider": "hive",...

seems we could close it, @danhuawang WDYT?

@caican00 , sorry for the delay, this PR is related to API which we should check it carefully.

> gently ping @Clearvive, could you help review this PR when you are free? Thank you @caican00 , Clearvive if not focused on Gravitino now

> > > gently ping @Clearvive, could you help review this PR when you are free? Thank you > > > > > > @caican00 , Clearvive if not focused...

> @FANNG1 could you please help review this PR? Thank you! ok