refactor: remove unused code about COLUMN metadata object

Author: unknowntpo

core/src/main/java/org/apache/gravitino/storage/relational/mapper/TableColumnMapper.java

@@ -36,9 +36,6 @@ public interface TableColumnMapper {
   List<ColumnPO> listColumnPOsByTableIdAndVersion(
       @Param("tableId") Long tableId, @Param("tableVersion") Long tableVersion);
 
-  @SelectProvider(type = TableColumnSQLProviderFactory.class, method = "listColumnPOsByColumnIds")
-  List<ColumnPO> listColumnPOsByColumnIds(@Param("columnIds") List<Long> columnIds);
-
   @InsertProvider(type = TableColumnSQLProviderFactory.class, method = "insertColumnPOs")
   void insertColumnPOs(@Param("columnPOs") List<ColumnPO> columnPOs);
 

core/src/main/java/org/apache/gravitino/storage/relational/mapper/TableColumnSQLProviderFactory.java

@@ -57,10 +57,6 @@ public static String listColumnPOsByTableIdAndVersion(
     return getProvider().listColumnPOsByTableIdAndVersion(tableId, tableVersion);
   }
 
-  public static String listColumnPOsByColumnIds(@Param("columnIds") List<Long> columnIds) {
-    return getProvider().listColumnPOsByColumnIds(columnIds);
-  }
-
   public static String insertColumnPOs(@Param("columnPOs") List<ColumnPO> columnPOs) {
     return getProvider().insertColumnPOs(columnPOs);
   }

core/src/main/java/org/apache/gravitino/storage/relational/mapper/provider/base/TableColumnBaseSQLProvider.java

@@ -47,28 +47,6 @@ public String listColumnPOsByTableIdAndVersion(
         + " ON t1.column_id = t2.column_id AND t1.table_version = t2.max_table_version";
   }
 
-  public String listColumnPOsByColumnIds(@Param("columnIds") List<Long> columnIds) {
-    return "<script>"
-        + " SELECT column_id AS columnId, column_name AS columnName,"
-        + " column_position AS columnPosition,"
-        + " metalake_id AS metalakeId, catalog_id AS catalogId,"
-        + " schema_id AS schemaId, table_id AS tableId,"
-        + " table_version AS tableVersion, column_type AS columnType,"
-        + " column_comment AS columnComment, column_nullable AS nullable,"
-        + " column_auto_increment AS autoIncrement,"
-        + " column_default_value AS defaultValue, column_op_type AS columnOpType,"
-        + " deleted_at AS deletedAt, audit_info AS auditInfo"
-        + " FROM "
-        + TableColumnMapper.COLUMN_TABLE_NAME
-        + " WHERE deleted_at = 0"
-        + " AND column_id in ("
-        + "<foreach collection='columnIds' item='columnId' separator=','>"
-        + "#{columnId}"
-        + "</foreach>"
-        + ") "
-        + "</script>";
-  }
-
   public String insertColumnPOs(@Param("columnPOs") List<ColumnPO> columnPOs) {
     return "<script>"
         + "INSERT INTO "

core/src/main/java/org/apache/gravitino/storage/relational/service/MetadataObjectService.java

@@ -31,7 +31,6 @@
 import org.apache.gravitino.storage.relational.mapper.MetalakeMetaMapper;
 import org.apache.gravitino.storage.relational.mapper.ModelMetaMapper;
 import org.apache.gravitino.storage.relational.mapper.SchemaMetaMapper;
-import org.apache.gravitino.storage.relational.mapper.TableColumnMapper;
 import org.apache.gravitino.storage.relational.mapper.TableMetaMapper;
 import org.apache.gravitino.storage.relational.mapper.TopicMetaMapper;
 import org.apache.gravitino.storage.relational.po.CatalogPO;
@@ -432,48 +431,6 @@ public static Map<Long, String> getTopicObjectsFullName(List<Long> topicIds) {
     return topicIdAndNameMap;
   }
 
-  /**
-   * Retrieves a map of Column object IDs to their full names.
-   *
-   * @param columnIds A list of Column object IDs to fetch names for.
-   * @return A Map where the key is the Column ID and the value is the Column full name. The map may
-   *     contain null values for the names if its parent object is deleted. Returns an empty map if
-   *     no Column objects are found for the given IDs. {@code @example} value of column full name:
-   *     "catalog1.schema1.table1.column1"
-   */
-  public static Map<Long, String> getColumnObjectsFullName(List<Long> columnIds) {
-    List<ColumnPO> columnPOs =
-        SessionUtils.getWithoutCommit(
-            TableColumnMapper.class, mapper -> mapper.listColumnPOsByColumnIds(columnIds));
-
-    if (columnPOs == null || columnPOs.isEmpty()) {
-      return new HashMap<>();
-    }
-
-    List<Long> tableIds = columnPOs.stream().map(ColumnPO::getTableId).collect(Collectors.toList());
-
-    Map<Long, String> tableIdAndNameMap = getTableObjectsFullName(tableIds);
-
-    HashMap<Long, String> columnIdAndNameMap = new HashMap<>();
-
-    columnPOs.forEach(
-        columnPO -> {
-          // since the table can be deleted, we need to check the
-          // null value, and when table is deleted, we will set fullName of columnPO to null
-          String tableName = tableIdAndNameMap.getOrDefault(columnPO.getTableId(), null);
-          if (tableName == null) {
-            LOG.warn("The table of column {} may be deleted", columnPO.getColumnId());
-            columnIdAndNameMap.put(columnPO.getColumnId(), null);
-            return;
-          }
-
-          String fullName = DOT_JOINER.join(tableName, columnPO.getColumnName());
-          columnIdAndNameMap.put(columnPO.getColumnId(), fullName);
-        });
-
-    return columnIdAndNameMap;
-  }
-
   /**
    * Retrieves a map of Catalog object IDs to their full names.
    *