Prevent Hive from returning table handles for Hudi tables

[homar]

Description

Additional context and related issues

Release notes

( ) This is not user-visible or docs only and no release notes are required.
( ) Release notes are required, please propose a release note for me.
( ) Release notes are required, with the following suggested text:

# Section
* Fix some things. ({issue}`issuenumber`)

[findepi]

/test-with-secrets sha=3fbb7a66c17124359eb5bb392ac84066372a445f

[homar]

@findepi I guess we can merge, cant we ?

[colebow]

Does this need release notes? cc @findepi

[findepi]

@colebow nope

[rahil-c]

@homar @findepi

Hi all, just wanted to get context for this change? Just wanted to confirm my understanding but if customers/users want to use Trino with hudi via the trino-hive connector will this change break them in newer releases, and their only option is to use trino-hudi connector?

cc @codope

[findepi]

i don't think trino-hive connector can be used to read Hudi tables. For instance, I believe it lacks necessary libraries on its classpath

[codope]

@findepi Hudi tables are queryable using the Hive connector provided users have the hudi presto/trino bundle under the hive plugin directory. This is done through the custom input format integration. This change has been there even before Trino was forked off Presto.
This patch will be a breaking change for those who use the Hive connector to query Hudi tables. Is it possible to revert this change?

[vinothchandar]

Landing this without us having the necessary migration path to hudi-connector, will cause pain & break production queries for users out there. @bvaradar this is going to break production at Robinhood.

We are still waiting on/working through PR reviews there to get hudi-connector on parity.

[vinothchandar]

cc @electrum Can we please be notified about such breaking changes in the future, so we can be part of the conversation.

[vburenin]

@homar Trino-hudi connector is not efficient enough and can't trim the search scope using metadata or metastore efficiently resulting in basically a full table scan. We have been using hive-connector for a long time to read hudi tables and it just works for us. I had to undo this change on our side to get things back to normal.