the-other-tim-brown commented on code in PR #13614:
URL: https://github.com/apache/hudi/pull/13614#discussion_r2229943986
##########
hudi-common/src/main/java/org/apache/hudi/common/table/read/BufferedRecordMergerFactory.java:
##########
@@ -402,6 +402,8 @@ public CustomPayloadBufferedRecordMerger(
Schema readerSchema,
TypedProperties props) {
super(readerContext, recordMerger, readerSchema, props);
+ ValidationUtils.checkArgument(!Objects.equals(payloadClass,
OverwriteWithLatestAvroPayload.class.getCanonicalName())
Review Comment:
Yes, this was happening per call previously so just moved it up to the
constructor. I think this is some defensive check to ensure that those payload
paths instead use the merge mode path. At this point I think we can remove this
check, what do you think?
--
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]
