dilipbiswal commented on pull request #2017: URL: https://github.com/apache/iceberg/pull/2017#issuecomment-754112570
@aokolnychyi Nice explanation about `early rewrite` in Analyzer. One other thing i worry about is, sometimes introduction of newer plan nodes may come in the way of existing optimization rules like pushing down filters or collapsing projects. Having said this, we may be okay in this case of `delete` in the present form. But we may have to worry about the future as the rule evolves. ---------------------------------------------------------------- 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]
