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 2021/06/07 11:32:18 UTC

[GitHub] [hudi] nsivabalan commented on pull request #2666: [HUDI-1160] Support update partial fields for CoW table

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:
users@infra.apache.org