andygrove opened a new pull request, #3854: URL: https://github.com/apache/datafusion-comet/pull/3854
## Which issue does this PR close? N/A - adds test coverage. ## Rationale for this change Add SQL file test coverage for casting double to string, specifically covering the negative zero (`-0.0`) use case to ensure Comet matches Spark behavior. Both Spark and Comet correctly cast `-0.0` to `"0.0"`. ## What changes are included in this PR? Adds `cast_double_to_string.sql` SQL file test covering: - Negative zero (`-0.0`) and positive zero (`0.0`) - Regular values (`1.5`, `-1.5`) - Special values (`NaN`, `Infinity`, `-Infinity`, `NULL`) - Scientific notation (`1.0E20`, `1.0E-20`, `-1.0E20`) - Small decimals (`0.001`) - Large values (`123456789.0`, `1.23456789E10`) Results are ordered by an integer `id` column rather than the double column to avoid ordering incompatibilities. ## How are these changes tested? This PR is a test. Verified passing with both `parquet.enable.dictionary=false` and `parquet.enable.dictionary=true` via ConfigMatrix. -- 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]
