You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@hudi.apache.org by GitBox <gi...@apache.org> on 2022/01/24 04:37:45 UTC

[GitHub] [hudi] nsivabalan commented on pull request #4660: [HUDI-3291][WIP][DO_NOT_MERGE] Flipping default record payload to DefaultHoodieRecordPayload

nsivabalan commented on pull request #4660:
URL: https://github.com/apache/hudi/pull/4660#issuecomment-1019712898


   @danny0405 : Can we please take up flink in another patch. Even for spark, I need to think through quite a bit before we can flip. I created this just to check if all tests pass for now. 
   For eg, in case of DefaultHoodieRecordPayload we expect payload props to be set. We do take the preCombine field and set the appropriate payload property. But wanted to follow up compaction, deltastreamer flows to ensure all paths are intact. 
   So, even in spark we have some work to be done. would prefer to keep flink in a separate PR. 
   


-- 
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: commits-unsubscribe@hudi.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org