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 2020/12/09 07:55:26 UTC

[GitHub] [hudi] nsivabalan edited a comment on pull request #2311: [HUDI-115] Adding OverwriteWithLatestAvroPayloadV1 to honor ordering with combineAndGetUpdateValue

nsivabalan edited a comment on pull request #2311:
URL: https://github.com/apache/hudi/pull/2311#issuecomment-741600626


   @vinothchandar : can you take a look at this patch. I am yet to find a good name for the new class added (OverwriteWithLatestAvroPayloadV1). Ignoring that for now, basically I have introduced a new config called "hoodie.datasource.honor.ordering.while.merging". Based on the value set, either existing OverwriteWithLatestAvroPayload will be used or the new class will be used while instantiating HoodiePayloads. By this way, callers of combineAndGetUpdateValue don't need to pass around the properties. 
   
   


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