Merge pull request #78 from jainlavina/lajain_unused_fs_fields

Remove unused fields
linkedin · Apr 20, 2024 · 8312af7 · 8312af7
2 parents ac1cb7e + 02336cc
commit 8312af7
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 6 deletions.
diff --git a/...ernalcatalog/src/main/java/com/linkedin/openhouse/internal/catalog/SnapshotInspector.java b/...ernalcatalog/src/main/java/com/linkedin/openhouse/internal/catalog/SnapshotInspector.java
@@ -1,6 +1,5 @@
 package com.linkedin.openhouse.internal.catalog;
 
-import com.linkedin.openhouse.cluster.storage.filesystem.FsStorageProvider;
 import com.linkedin.openhouse.internal.catalog.exception.InvalidIcebergSnapshotException;
 import java.io.UncheckedIOException;
 import java.util.List;
@@ -28,8 +27,6 @@
  */
 @Component
 public class SnapshotInspector {
-  @Autowired private FsStorageProvider fsStorageProvider;
-
   @Autowired private Consumer<Supplier<Path>> fileSecurer;
   /**
    * TODO: ADD Validation for snapshot: Sequence-number based, schema-id based, see iceberg spec for

diff --git a/...n/java/com/linkedin/openhouse/tables/repository/impl/OpenHouseInternalRepositoryImpl.java b/...n/java/com/linkedin/openhouse/tables/repository/impl/OpenHouseInternalRepositoryImpl.java
@@ -49,7 +49,6 @@
 import org.apache.iceberg.catalog.TableIdentifier;
 import org.apache.iceberg.exceptions.CommitFailedException;
 import org.apache.iceberg.exceptions.NoSuchTableException;
-import org.apache.iceberg.io.FileIO;
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.stereotype.Component;
 
@@ -74,8 +73,6 @@ public class OpenHouseInternalRepositoryImpl implements OpenHouseInternalReposit
 
   @Autowired FsStorageProvider fsStorageProvider;
 
-  @Autowired FileIO fileIO;
-
   @Autowired MeterRegistry meterRegistry;
 
   @Autowired SchemaValidator schemaValidator;