Remove table layouts from all connectors

[electrum]

Add a comment about which one you are working on. Each one should be done as a separate commit and PR, since they are unrelated, and some will be non-trivial or will require additional features like column projection pushdown.

• Accumulo (Remove TableLayout from Accumulo connector #823)
• Atop (Remove TableLayout from Atop connector #885)
• Cassandra (Remove TableLayout from Cassandra connector #1004)
• Elasticsearch (Remove TableLayout from Elasticsearch connector #883)
• Hive (Remove table layouts from Hive #689)
• Jmx (Migrate JMX connector to new APIs #511)
• Kafka (Remove TableLayout from Kafka connector #881)
• Kudu (Remove table layout APIs from Kudu Connector #1721)
• LocalFile (Remove TableLayout from Local file connector #880)
• Mongo (Remove TableLayout from Mongo connector #884)
• Phoenix (Remove TableLayout API from Phoenix connector #3383)
• Raptor (Remove TableLayout API from Raptor connector #1015)
• Redis (Remove TableLayout from Redis connector #882)
• Thrift (Remove TableLayout API from Thrift connector #2009)
• Tpcds (Remove TableLayout from TPCDS connector #9628)
• Tpch (Remove table layouts from TPCH connector #787)

[Praveen2112]

@electrum Can I take up this issue ?

[electrum]

@Praveen2112 Yes, but work on one at a time, and comment here first about which one you are doing. We can have multiple people work on these in parallel. Some of these will be non-trivial or may require additional features (like column projection pushdown which is not yet implemented in the new model).

[Praveen2112]

@electrum I will be starting with the Tpch and Tpcds connector

[Praveen2112]

I am working on removing table layouts from Accumulo connector.

[Praveen2112]

Next I am trying to remove for Atop connector.

[Praveen2112]

Next I'm planning to take cassandra and raptor connector.

[Praveen2112]

Once #676 is merged we can remove TableLayout for Kudu and Thrift connector

[findepi]

In the project i see TpchTableLayoutHandle and TpcdsTableLayoutHandle classes, so we're almost migrated.

Do we have a plan to do something with io.trino.spi.connector.ConnectorMetadata#usesLegacyTableLayouts?
It currently returns false.

Removing support for layouts would simplify things in the engine.

[martint]

There was some issue with the TPC-DS connector due to limitations of the optimizer. @Praveen2112, do you remember?

Once those are migrated, we can just go ahead and remove the old API. It’s been almost two years since we deprecated it.