Removing functions not needed.

Author: Arijit6258

extensions-contrib/druid-iceberg-extensions/src/main/java/org/apache/druid/iceberg/input/IcebergCatalog.java

@@ -169,21 +169,4 @@ public List<String> extractSnapshotDataFiles(
     }
     return dataFilePaths;
   }
-
-  public List<String> extractSnapshotDataFiles(
-          String tableNamespace,
-          String tableName,
-          IcebergFilter icebergFilter,
-          DateTime snapshotTime,
-          ResidualFilterMode residualFilterMode
-  ) {
-    return extractSnapshotDataFiles(
-            tableNamespace,
-            tableName,
-            icebergFilter,
-            snapshotTime,
-            residualFilterMode,
-            null
-    );
-  }
 }

extensions-contrib/druid-iceberg-extensions/src/test/java/org/apache/druid/iceberg/input/IcebergInputSourceTest.java

@@ -405,102 +405,6 @@ public void testResidualFilterModeFailWithPartitionedTableNonPartitionColumn() t
     );
   }
 
-  @Test
-  public void testInputSourceWithColumnProjection() throws IOException
-  {
-    IcebergInputSource inputSource = new IcebergInputSource(
-        TABLENAME,
-        NAMESPACE,
-        null,
-        testCatalog,
-        new LocalInputSourceFactory(),
-        null,
-        null
-    );
-    Stream<InputSplit<List<String>>> splits = inputSource.createSplits(null, new MaxSizeSplitHintSpec(null, null));
-    List<File> localInputSourceList = splits.map(inputSource::withSplit)
-                                            .map(inpSource -> (LocalInputSource) inpSource)
-                                            .map(LocalInputSource::getFiles)
-                                            .flatMap(List::stream)
-                                            .collect(Collectors.toList());
-
-    Assert.assertEquals(1, inputSource.estimateNumSplits(null, new MaxSizeSplitHintSpec(1L, null)));
-    Assert.assertEquals(1, localInputSourceList.size());
-
-    // First, read all columns without projection
-    List<Record> allRecords = new ArrayList<>();
-    CloseableIterable<Record> fullDataReader = Parquet.read(Files.localInput(localInputSourceList.get(0)))
-                                                      .project(tableSchema)
-                                                      .createReaderFunc(fileSchema -> GenericParquetReaders.buildReader(
-                                                          tableSchema,
-                                                          fileSchema
-                                                      ))
-                                                      .build();
-
-    for (Record record : fullDataReader) {
-      allRecords.add(record);
-    }
-    fullDataReader.close();
-
-    Assert.assertEquals(1, allRecords.size());
-    Record fullRecord = allRecords.get(0);
-    Assert.assertEquals(2, fullRecord.size()); // Should have 2 columns
-    Assert.assertEquals(tableData.get("id"), fullRecord.get(0));
-    Assert.assertEquals(tableData.get("name"), fullRecord.get(1));
-
-    // Read with projection - only 'id' column
-
-    // Create schema with only the 'id' column for projection
-    Schema idProjectedSchema = new Schema(
-            Types.NestedField.required(1, "id", Types.StringType.get())
-    );
-
-    List<Record> idProjectedRecords = new ArrayList<>();
-    CloseableIterable<Record> idProjectedReader = Parquet.read(Files.localInput(localInputSourceList.get(0)))
-                                                         .project(idProjectedSchema)
-                                                         .createReaderFunc(fileSchema -> GenericParquetReaders.buildReader(
-                                                             idProjectedSchema,
-                                                             fileSchema
-                                                         ))
-                                                         .build();
-
-    for (Record record : idProjectedReader) {
-      idProjectedRecords.add(record);
-    }
-    idProjectedReader.close();
-
-    Assert.assertEquals(1, idProjectedRecords.size());
-    Record idRecord = idProjectedRecords.get(0);
-    Assert.assertEquals(1, idRecord.size()); // Should have only 1 column
-    Assert.assertEquals(fullRecord.get(0), idRecord.get(0)); // Id should match full record
-
-    // Read with projection - only 'name' column
-
-    // Create schema with only the 'name' column for projection
-    Schema nameProjectedSchema = new Schema(
-            Types.NestedField.required(2, "name", Types.StringType.get())
-    );
-
-    List<Record> nameProjectedRecords = new ArrayList<>();
-    CloseableIterable<Record> nameProjectedReader = Parquet.read(Files.localInput(localInputSourceList.get(0)))
-                                                           .project(nameProjectedSchema)
-                                                           .createReaderFunc(fileSchema -> GenericParquetReaders.buildReader(
-                                                               nameProjectedSchema,
-                                                               fileSchema
-                                                           ))
-                                                           .build();
-
-    for (Record record : nameProjectedReader) {
-      nameProjectedRecords.add(record);
-    }
-    nameProjectedReader.close();
-
-    Assert.assertEquals(1, nameProjectedRecords.size());
-    Record nameRecord = nameProjectedRecords.get(0);
-    Assert.assertEquals(1, nameRecord.size()); // Should have only 1 column
-    Assert.assertEquals(fullRecord.get(1), nameRecord.get(0)); // Name should match full record
-  }
-
   @After
   public void tearDown()
   {