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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to