yupeng9 commented on PR #10702: URL: https://github.com/apache/pinot/pull/10702#issuecomment-1529840410
> Hi Team, reading with a neutral perspective, it seems like there were some miscommunications that led to duplicate effort on both sides. But I am 100% sure everyone here here: > > * Has the best interest of the Apache Pinot project before anything. > * Would benefit from this feature being completed as soon as possible. > > With above in mind, and the contribution guidelines, I do see that this [design doc ](https://docs.google.com/document/d/19s1AHCRjmqeVa0z_djYBTAt_meg6qWcUxwRO3y-G9O4/edit?usp=drivesdk) being reviewed. > > My recommendation would be for all of us to collaborate on that design and any associated PRs to take this feature to completion as soon as possible. > > Thank you all! #One Team. +1 to #One Team Also I hope we can agree on the direction. One key difference I can see between these two approaches is to reuse validDocId or not for deletion. I also left this comment on @navina's design. This is a key as it has implications on correctness/efficiency/complexity etc. Could you provide a detailed example on reusing validDocId is insufficient to help this discussion. Also, I'm curious what's the behavior if there's an update to record after deletion, will we ignore it or revive the record? -- 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-unsubscr...@pinot.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org
