rahil-c commented on code in PR #14313:
URL: https://github.com/apache/hudi/pull/14313#discussion_r2557038292


##########
hudi-common/src/main/java/org/apache/hudi/common/schema/HoodieSchema.java:
##########
@@ -718,6 +720,25 @@ public HoodieSchema parse(String jsonSchema) {
         throw new HoodieAvroSchemaException("Failed to parse schema: " + 
jsonSchema, e);
       }
     }
+
+    /**
+     * Parses a schema from an InputStream.
+     *
+     * @param inputStream the InputStream containing the JSON schema
+     * @return parsed HoodieSchema
+     * @throws java.io.IOException if reading from the stream fails
+     * @throws HoodieAvroSchemaException if the schema is invalid
+     */
+    public HoodieSchema parse(InputStream inputStream) throws 
HoodieIOException {

Review Comment:
   I think I misinterpreted your earlier comment 
https://github.com/apache/hudi/pull/14313#discussion_r2554407814.
   So to confirm is the pattern we want to follow something similar to the 
above parse method: 
https://github.com/apache/hudi/blob/master/hudi-common/src/main/java/org/apache/hudi/common/schema/HoodieSchema.java#L710
   
   If so then I am planning on making this change below
   <img width="1469" height="549" alt="Screenshot 2025-11-24 at 8 48 22 AM" 
src="https://github.com/user-attachments/assets/346a3743-8dc8-4695-b0c9-c62ba31925e5";
 />
   



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to