the-other-tim-brown commented on code in PR #13987:
URL: https://github.com/apache/hudi/pull/13987#discussion_r2388551168
##########
hudi-common/src/main/java/org/apache/hudi/common/table/read/FileGroupReaderSchemaHandler.java:
##########
@@ -74,6 +74,10 @@ public class FileGroupReaderSchemaHandler<T> {
// requiredSchema: the requestedSchema with any additional columns required
for merging etc
protected final Schema requiredSchema;
+ // the schema for incoming records, usually it equals with the
requiredSchema,
+ // the only exception is for COW merge cases and MOR small file handling,
which do not include the metadata fields.
+ protected Schema incomingRecordSchema;
Review Comment:
Maybe `updateRecordSchema`? In both cases the incoming data represents
updates to the base file's records
--
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]
