danny0405 commented on code in PR #13614:
URL: https://github.com/apache/hudi/pull/13614#discussion_r2229869570
##########
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:
why we need this? The merger should not care about specific impl of the
payload IMO, the check should be done before the merger is created.
--
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]
