Use new interface in tests

Author: XJDKC

quarkus/service/src/test/java/org/apache/polaris/service/quarkus/catalog/BasePolarisCatalogTest.java

@@ -1541,23 +1541,24 @@ public void testFileIOWrapper() {
         .isGreaterThan(0);
 
     Assertions.assertThat(catalog.dropTable(TABLE)).as("Table deletion should succeed").isTrue();
+    TaskEntity taskEntity =
+        TaskEntity.of(
+            metaStoreManager
+                .loadTasks(metaStoreSession, "testExecutor", 1)
+                .getEntities()
+                .getFirst());
+    Map<String, String> properties = taskEntity.getInternalPropertiesAsMap();
+    properties.put(CatalogProperties.FILE_IO_IMPL, "org.apache.iceberg.inmemory.InMemoryFileIO");
+    taskEntity.setInternalPropertiesAsMap(properties);
     TableCleanupTaskHandler handler =
         new TableCleanupTaskHandler(
             Mockito.mock(),
             createMockMetaStoreManagerFactory(),
             configurationStore,
             diagServices,
-            (task, rc) ->
-                measured.loadFileIO(
-                    realmId, "org.apache.iceberg.inmemory.InMemoryFileIO", Map.of()),
+            new TaskFileIOSupplier(measured),
             clock);
-    handler.handleTask(
-        TaskEntity.of(
-            metaStoreManager
-                .loadTasks(metaStoreSession, "testExecutor", 1)
-                .getEntities()
-                .getFirst()),
-        realmId);
+    handler.handleTask(taskEntity, realmId);
     Assertions.assertThat(measured.getNumDeletedFiles()).as("A table was deleted").isGreaterThan(0);
   }
 }

service/common/src/main/java/org/apache/polaris/service/catalog/io/FileIOFactory.java

@@ -35,19 +35,7 @@
  */
 public interface FileIOFactory {
 
-  /**
-   * Loads a FileIO implementation for the specified realm and class name with the provided
-   * properties.
-   *
-   * <p>This method is commonly used by TaskFileIOSupplier and Tests to instantiate a FileIO object
-   * based on the provided configuration. Usually, the properties will contain the full info to
-   * instantiate a FileIO.
-   *
-   * @param realmId the identifier of the realm for which the FileIO is being loaded.
-   * @param ioImplClassName the fully qualified class name of the FileIO implementation to load.
-   * @param properties a map of configuration properties to initialize the FileIO implementation.
-   * @return an instance of the FileIO implementation configured with the specified properties.
-   */
+  /** This method is intended for use in tests only. */
   FileIO loadFileIO(
       @Nonnull RealmId realmId,
       @Nonnull String ioImplClassName,