Addressed comments

Author: voonhous

hudi-hadoop-common/src/main/java/org/apache/hudi/common/util/AvroOrcUtils.java

@@ -353,7 +353,7 @@ public static void addToVector(TypeDescription type, ColumnVector colVector, Hoo
    *
    * @param unionVector       The vector to store value.
    * @param unionChildTypes   All possible types for the value Object.
-   * @param schema            Avro union schema for the value Object.
+   * @param schema            Union schema for the value Object.
    * @param value             Object to be added to the unionVector
    * @param vectorPos         The position in the vector where value will be stored at.
    * @return                  succeeded or failed
@@ -801,7 +801,7 @@ public static HoodieSchema createSchema(TypeDescription orcSchema) {
    * the nullability of an Avro type. To achieve consistency between the Avro and ORC schema,
    * non-NULL types are extracted from the union type.
    * @param unionSchema       A schema of union type.
-   * @return  An Avro schema that is either NULL or a UNION without NULL fields.
+   * @return  A schema that is either NULL or a UNION without NULL fields.
    */
   private static HoodieSchema getActualSchemaType(HoodieSchema unionSchema) {
     final List<HoodieSchema> nonNullMembers = unionSchema.getTypes().stream()

hudi-hadoop-common/src/main/java/org/apache/hudi/io/storage/hadoop/HoodieAvroParquetReader.java

@@ -112,7 +112,6 @@ protected ClosableIterator<IndexedRecord> getIndexedRecordIterator(HoodieSchema
 
   @Override
   public ClosableIterator<IndexedRecord> getIndexedRecordIterator(HoodieSchema readerSchema, HoodieSchema requestedSchema) throws IOException {
-    //TODO boundary for now to revisit in later pr to use HoodieSchema
     return getIndexedRecordIteratorInternal(requestedSchema, Collections.emptyMap());
   }
 
@@ -123,8 +122,13 @@ public ClosableIterator<IndexedRecord> getIndexedRecordIterator(HoodieSchema rea
 
   @Override
   public HoodieSchema getSchema() {
-    fileSchema = fileSchema.or(() -> Option.ofNullable(parquetUtils.readSchema(storage, path)));
-    return fileSchema.get();
+    // Lazy initialization with caching: read schema from parquet file footer on first call,
+    // then cache it in fileSchema to avoid repeated I/O on subsequent calls
+    return fileSchema.orElseGet(() -> {
+      HoodieSchema schema = parquetUtils.readSchema(storage, path);
+      fileSchema = Option.ofNullable(schema);
+      return schema;
+    });
   }
 
   @Override