the-other-tim-brown commented on code in PR #13742:
URL: https://github.com/apache/hudi/pull/13742#discussion_r2308933595
##########
hudi-common/src/main/java/org/apache/hudi/common/model/HoodieAvroRecordMerger.java:
##########
@@ -19,51 +19,101 @@
package org.apache.hudi.common.model;
import org.apache.hudi.common.config.TypedProperties;
+import org.apache.hudi.common.engine.RecordContext;
import org.apache.hudi.common.model.HoodieRecord.HoodieRecordType;
+import org.apache.hudi.common.table.read.BufferedRecord;
+import org.apache.hudi.common.table.read.BufferedRecords;
+import org.apache.hudi.common.util.ConfigUtils;
+import org.apache.hudi.common.util.HoodieRecordUtils;
import org.apache.hudi.common.util.Option;
-import org.apache.hudi.common.util.collection.Pair;
import org.apache.avro.Schema;
+import org.apache.avro.generic.GenericRecord;
import org.apache.avro.generic.IndexedRecord;
import java.io.IOException;
-import java.util.Properties;
/**
* Record merger for Hoodie avro record.
*
* <p>It should only be used for base record from disk to merge with incoming
record.
*/
public class HoodieAvroRecordMerger implements HoodieRecordMerger,
OperationModeAwareness {
- public static final HoodieAvroRecordMerger INSTANCE = new
HoodieAvroRecordMerger();
+ public static final String PAYLOAD_CLASS_PROP =
"_hoodie.merger.payload.class";
Review Comment:
Yes, since it will differ from the table's payload class. Not sure what the
best way is to ensure the merger has the right payload class set during the
merge into operations.
--
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]
