pengzhiwei2018 commented on a change in pull request #2645: URL: https://github.com/apache/hudi/pull/2645#discussion_r618345744
########## File path: hudi-client/hudi-client-common/src/main/java/org/apache/hudi/io/HoodieMergeHandle.java ########## @@ -278,11 +279,17 @@ public void write(GenericRecord oldRecord) { HoodieRecord<T> hoodieRecord = new HoodieRecord<>(keyToNewRecords.get(key)); try { Option<IndexedRecord> combinedAvroRecord = - hoodieRecord.getData().combineAndGetUpdateValue(oldRecord, useWriterSchema ? writerSchemaWithMetafields : writerSchema, + hoodieRecord.getData().combineAndGetUpdateValue(oldRecord, + useWriterSchema ? inputSchemaWithMetaFields : inputSchema, Review comment: yeah, the `HoodiePayload` use the `inputSchema` to parse the incoming record and compute the result record. The result record will have the full table schema. -- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org