danny0405 commented on code in PR #13519:
URL: https://github.com/apache/hudi/pull/13519#discussion_r2218404818
##########
hudi-common/src/main/java/org/apache/hudi/common/table/HoodieTableConfig.java:
##########
@@ -803,6 +810,10 @@ public String getPayloadClass() {
return HoodieRecordPayload.getPayloadClassName(this);
}
+ public String getLegacyPayloadClass() {
+ return getStringOrDefault(LEGACY_PAYLOAD_CLASS_NAME, EMPTY_STRING);
Review Comment:
The payload class is still used in write path for avro record type, even for
Spark engine, the default record type is still AVRO.
We might need to introduce a new `HoodieRecord` impl for avro record type to
take the avro bytes, for e.g: `HoodieAvroBinaryRecord`, so that it can be used
to replaced the payload based avro record write path.
--
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]
