danny0405 edited a comment on pull request #4880:
URL: https://github.com/apache/hudi/pull/4880#issuecomment-1058877440


   > Looked at the code, seems like we do queue up the log blocks in order and 
for DMS/Debezium payloads we pass the right ordering field value., extending 
`OverwriteWithLatestAvroPayload`. So it should all work.
   
   I see that the `AbstractDebeziumAvroPayload` extends 
`OverwriteWithLatestAvroPayload` directly and it just nullify the payload bytes 
for the DELETE records, these records would then write into the DELETE block 
too so the bug still exists.
   
   > Whats the payload used for Flink CDC? cc @danny0405 
   
   The default payload.
   
   I have addressed the review comments, please review again ~


-- 
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]


Reply via email to