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


   @vinothchandar : my thoughts on adding this payload support for MOR. 
   It may not make sense to add this payload support to MOR. bcoz, in MOR we 
don't lookup previous record values as we only append records to logs. But this 
payload involves fetching previous record and merging with incoming one. So, 
might as well end up using COW for simplicity and since MOR is mainly used to 
shave off some latency during writes. 
   Open to hear your thoughts. 


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

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


Reply via email to