From c18df8131417975154de258270984e809435157d Mon Sep 17 00:00:00 2001
From: Piotr Findeisen <piotr.findeisen@gmail.com>
Date: Wed, 9 Aug 2023 07:29:21 +0200
Subject: [PATCH] Deprecate getTableHandle without versions

It got superseded by an overload that takes versions in
2a466edc37e26047d8cbc7f98023aa62452e782c. It should be deprecated since
then. Some connectors (like Iceberg) do not implement the version-less
overload at all.
---
 .../src/main/java/io/trino/spi/connector/ConnectorMetadata.java | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/core/trino-spi/src/main/java/io/trino/spi/connector/ConnectorMetadata.java b/core/trino-spi/src/main/java/io/trino/spi/connector/ConnectorMetadata.java
index 898abbe2a285..59427dc46316 100644
--- a/core/trino-spi/src/main/java/io/trino/spi/connector/ConnectorMetadata.java
+++ b/core/trino-spi/src/main/java/io/trino/spi/connector/ConnectorMetadata.java
@@ -104,8 +104,10 @@ default List<String> listSchemaNames(ConnectorSession session)
      * cannot be queried.
      * @see #getView(ConnectorSession, SchemaTableName)
      * @see #getMaterializedView(ConnectorSession, SchemaTableName)
+     * @deprecated Implement {@link #getTableHandle(ConnectorSession, SchemaTableName, Optional, Optional)}.
      */
     @Nullable
+    @Deprecated
     default ConnectorTableHandle getTableHandle(ConnectorSession session, SchemaTableName tableName)
     {
         return null;