You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pinot.apache.org by "deemoliu (via GitHub)" <gi...@apache.org> on 2023/07/25 21:40:22 UTC

[GitHub] [pinot] deemoliu opened a new issue, #11177: [upsert] partial upsert interface enhancement

deemoliu opened a new issue, #11177:
URL: https://github.com/apache/pinot/issues/11177

   Discussed with @Jackie-Jiang on the following improvement on partial upsert inteface
   
   - move partial upsert merge logic to partial upsert handler 
   - optimize partial upsert to avoid read values if value is null
   
   With the above change, we can read the read access for unnecessary fields. Also it will be helpful for future extensions e.g. #11174
   
   


-- 
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: commits-unsubscribe@pinot.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org


[GitHub] [pinot] deemoliu commented on issue #11177: [upsert] partial upsert interface enhancement

Posted by "deemoliu (via GitHub)" <gi...@apache.org>.
deemoliu commented on issue #11177:
URL: https://github.com/apache/pinot/issues/11177#issuecomment-1678273135

   Linked PR: https://github.com/apache/pinot/pull/10927


-- 
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: commits-unsubscribe@pinot.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org


[GitHub] [pinot] Jackie-Jiang closed issue #11177: [upsert] partial upsert interface enhancement

Posted by "Jackie-Jiang (via GitHub)" <gi...@apache.org>.
Jackie-Jiang closed issue #11177: [upsert] partial upsert interface enhancement
URL: https://github.com/apache/pinot/issues/11177


-- 
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: commits-unsubscribe@pinot.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org


[GitHub] [pinot] deemoliu commented on issue #11177: [upsert] partial upsert interface enhancement

Posted by "deemoliu (via GitHub)" <gi...@apache.org>.
deemoliu commented on issue #11177:
URL: https://github.com/apache/pinot/issues/11177#issuecomment-1650599566

   @Jackie-Jiang i am working on this on and will clean up the PR this week.


-- 
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: commits-unsubscribe@pinot.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org