luoyuxia commented on issue #7311: URL: https://github.com/apache/iceberg/issues/7311#issuecomment-1556786949
> > Hi, just for reference, Pamion has integrated [UPDATE](https://github.com/apache/incubator-paimon/pull/1010) & [DELETE](https://github.com/apache/incubator-paimon/issues/786) for flink-1.17. Also in previous investagtion, IIUC, to implement DELETE & UPDATE for iceberg, we may need to finish #7110 first since Iceberg will use the metadata columns, file_path, file_pos to delete data. > > The meta columns are required for UPDATE & DELETE on tables without pk. But for tables with pk, we can UPDATE & DELETE using the pk, which does not rely on #7110. In fact, UPDATE & DELETE in Paimon also only supports tables with pk. > > IMHO, we can first support tables with pk. While support for tables without pk can be added later, after #7110 is finished. @luoyuxia @clownxc What are your thoughts on this? Thanks! > > cc @hililiwei @stevenzwu @chenjunjiedada Sounds good to me. -- 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]
