nsivabalan commented on code in PR #13964: URL: https://github.com/apache/hudi/pull/13964#discussion_r2370359635
########## hudi-common/src/main/java/org/apache/hudi/common/model/SerializableIndexedRecord.java: ########## @@ -46,6 +46,7 @@ public class SerializableIndexedRecord implements GenericRecord, KryoSerializabl private static final long serialVersionUID = 1L; private IndexedRecord record; + private Schema schema; Review Comment: Note: for now, we wanted to see end to end all writing paths w/ this solution, where we have schema stored per record just within write handles. Once we have full handle to all write paths, and fished out all thats required for lazy deser, we wanted to optimize this to see if we can remove `schema` and get it working just w/ schemaId. For now, I can think of creating a HoodieRecord from BufferedRecord and then calling decode(Schema) which does not seem the right way to go. But will come back to this. -- 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]
