srielau commented on PR #58057: URL: https://github.com/apache/spark/pull/58057#issuecomment-5322299210
Closing in favor of #58033. These are tests that verify the foundation change, not a change of their own: the single-pass resolver has no CHAR/VARCHAR-specific logic, so parity under `spark.sql.charVarchar.standardSemantics.enabled` follows from `Expression.dataType` and shared `TypeCoercion` in #58033. Reviewing the matrix apart from the code it exercises made both harder to judge, so the coverage now lives alongside that code and SPARK-58802 is satisfied there. -- 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]
