himanish-star commented on issue #14924: URL: https://github.com/apache/pinot/issues/14924#issuecomment-2680769117
> [@himanish-star](https://github.com/himanish-star) please go with solution2 because the current behavior for "Overwrite" merger is an expected behavior in many use cases. Using different mergers "Overwrite" and "ForceOverwrite" can provide the flexibility for users to select how to handle null values. > > [@raghukn](https://github.com/raghukn) one of the most common use case of partial upsert is to fill the missing field values for a existing row (primary key). Solution1 will break it. Hi @deemoliu , I've updated the PR with approach 2 and I've also added unit tests. I'll update the docs once the PR is checked-in. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
