rdblue commented on pull request #1947: URL: https://github.com/apache/iceberg/pull/1947#issuecomment-749119861
> About count implementation Anton, i was thinking to implement it without the optimization as a first cut and optimize it in a follow-up. If I understand correctly, it is actually easier to do the optimization now because the optimization only requires changes in merge-on-read. Anton said this: > We are already doing this inner join for copy-on-write to find matches so we can modify that code to also do the cardinality check at the same time. Since we are currently only implementing copy-on-write, I think it will be easier to do the cardinality check in the existing inner join. ---------------------------------------------------------------- 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: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
