dilipbiswal commented on pull request #1947: URL: https://github.com/apache/iceberg/pull/1947#issuecomment-749143069
@rdblue > 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. Oh.. since we have two options to choose from and were discussing which option to choose.. i thought doing a count check as a side thing (basically does the join twice) and raise an error as a start. But if we can pick one option between the two proposals now, i can give a try to implement it. ---------------------------------------------------------------- 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]
