yupeng9 commented on pull request #6153: URL: https://github.com/apache/incubator-pinot/pull/6153#issuecomment-712380883
@Jackie-Jiang I think the current implementation assumes the primary key cannot be the time column. Also, in reality, it does not quite make sense to use the time column as the primary key? If the intention is to use a later record to overwrite a previous one with the same timestamp, then it won't work for Pinot as the update behavior of the records with same timestamp is undefined. ---------------------------------------------------------------- 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: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org