fix: remove column and model

Author: unknowntpo

api/src/main/java/org/apache/gravitino/authorization/SecurableObjects.java

@@ -88,22 +88,6 @@ public static SecurableObject ofTable(
     return of(MetadataObject.Type.TABLE, names, privileges);
   }
 
-  /**
-   * Create the column {@link SecurableObject} with the given securable schema object, column name
-   * and privileges.
-   *
-   * @param table The table securable object
-   * @param column The column name
-   * @param privileges The privileges of the column
-   * @return The created column {@link SecurableObject}
-   */
-  public static SecurableObject ofColumn(
-      SecurableObject table, String column, List<Privilege> privileges) {
-    List<String> names = Lists.newArrayList(DOT_SPLITTER.splitToList(table.fullName()));
-    names.add(column);
-    return of(MetadataObject.Type.TABLE, names, privileges);
-  }
-
   /**
    * Create the topic {@link SecurableObject} with the given securable schema object ,topic name and
    * privileges.
@@ -136,22 +120,6 @@ public static SecurableObject ofFileset(
     return of(MetadataObject.Type.FILESET, names, privileges);
   }
 
-  /**
-   * Create the table {@link SecurableObject} with the given securable schema object, fileset name
-   * and privileges.
-   *
-   * @param schema The schema securable object
-   * @param model The model name
-   * @param privileges The privileges of the model
-   * @return The created model {@link SecurableObject}
-   */
-  public static SecurableObject ofModel(
-      SecurableObject schema, String model, List<Privilege> privileges) {
-    List<String> names = Lists.newArrayList(DOT_SPLITTER.splitToList(schema.fullName()));
-    names.add(model);
-    return of(MetadataObject.Type.MODEL, names, privileges);
-  }
-
   private static class SecurableObjectImpl extends MetadataObjectImpl implements SecurableObject {
 
     private List<Privilege> privileges;

core/src/test/java/org/apache/gravitino/storage/relational/service/TestSecurableObjects.java

@@ -34,7 +34,6 @@
 import org.apache.gravitino.meta.BaseMetalake;
 import org.apache.gravitino.meta.CatalogEntity;
 import org.apache.gravitino.meta.FilesetEntity;
-import org.apache.gravitino.meta.ModelEntity;
 import org.apache.gravitino.meta.RoleEntity;
 import org.apache.gravitino.meta.SchemaEntity;
 import org.apache.gravitino.meta.TableEntity;
@@ -97,17 +96,6 @@ public void testAllTypeSecurableObjects() throws IOException {
             auditInfo);
     backend.insert(topic, false);
 
-    ModelEntity model =
-        createModelEntity(
-            RandomIdGenerator.INSTANCE.nextId(),
-            Namespace.of("metalake", "catalog", "schema"),
-            "model",
-            "model comment",
-            1,
-            ImmutableMap.of("key", "value"),
-            auditInfo);
-    backend.insert(model, false);
-
     SecurableObject metalakeObject =
         SecurableObjects.ofMetalake(
             metalake.name(), Lists.newArrayList(Privileges.UseCatalog.allow()));
@@ -137,10 +125,6 @@ public void testAllTypeSecurableObjects() throws IOException {
         SecurableObjects.ofTopic(
             schemaObject, "topic", Lists.newArrayList(Privileges.ConsumeTopic.deny()));
 
-    SecurableObject modelObject =
-        SecurableObjects.ofModel(
-            schemaObject, "model", Lists.newArrayList(Privileges.ConsumeTopic.deny()));
-
     ArrayList<SecurableObject> securableObjects =
         Lists.newArrayList(
             metalakeObject,
@@ -149,7 +133,6 @@ public void testAllTypeSecurableObjects() throws IOException {
             schemaObject,
             tableObject,
             filesetObject,
-            modelObject,
             topicObject);
     securableObjects.sort(Comparator.comparing(MetadataObject::fullName));