vrozov commented on PR #53121: URL: https://github.com/apache/spark/pull/53121#issuecomment-3576486833
@cloud-fan I don't agree with your reasoning, as I already mentioned several times in my comments: https://github.com/apache/spark/pull/53121#discussion_r2542669548, https://github.com/apache/spark/pull/53121#discussion_r2547703107. Somehow, it was important for you to discuss this change, and https://github.com/apache/spark/blob/d14209c6ffba991b1d8ca9580708dea1ee37920e/sql/core/src/test/resources/sql-tests/inputs/identifier-clause.sql#L321 does not matter. Consider `identifier-clause.sql` (not a legacy case). Will you use `IF EXIST` for the view that is created in the same script? Anyway, if it is important for you and the community to have less "noise", I would expect other similar cases where a table or view fails to be created and then dropped to be updated to use `IF EXIST` at least to be consistent and avoid confusion from other committers. -- 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]
