viirya commented on PR #36402: URL: https://github.com/apache/spark/pull/36402#issuecomment-1112880149
It looks good to me. Just a bit confused by: > However, the new analyzer rule RewriteDeleteFromTable fails very early if the v2 table does not support delete. This breaks certain Spark extensions which can still execute DeleteFromTable for certain v2 tables. So, `RewriteDeleteFromTable` fails if the v2 table does not support delete. Why Spark extensions still can execute `DeleteFromTable` for them, if they don't support? -- 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]
