[GitHub] [incubator-pinot] jamesyfshao commented on issue #5175: add pinot upsert features to pinot common

[GitBox <gi...@apache.org>]

jamesyfshao commented on issue #5175: add pinot upsert features to pinot common
URL: https://github.com/apache/incubator-pinot/pull/5175#issuecomment-603604130
 
 
   > Can you please link the design doc in the pr description so that it is easier to understand the concept of upsert?
   > Since this pr is touching the core configs, we need to spend some time discussing how to design the config keys because they are very hard to change in the future
   
   Linked the design doc in the description. also [google doc design doc](https://www.google.com/url?sa=t&rct=j&esrc=s&source=appssearch&uact=8&cd=0&cad=rja&q&sig2=4ypfJoFtqjSdua_7dziWCQ&ved=0ahUKEwjVhpqIybToAhXDHWIKHWFcBVs4ABABKAAwAA&url=https://drive.google.com/a/uber.com/open?id%3D1SFFir7ByxCff-aVYxQeTHpNhPXeP5q7P4g_6O2iNGgU%26usp%3Dchrome_omnibox&usg=AOvVaw3KEdQYFMPa7tgpOn01R5ol) with comments. We have discussed this a long time back but I definitely open to go through it again as things have already changed. Do you think we should do some remote meeting sync or something else?

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


With regards,
Apache Git Services

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