sunchao commented on PR #5420: URL: https://github.com/apache/datafusion-comet/pull/5420#issuecomment-5448633936
Fixed both re-review findings in [97f605ede](https://github.com/apache/datafusion-comet/commit/97f605ede8120fa354371baaddb98cdc3cfb3c10). The six affected TPC-DS plan expectations were regenerated with the supported harness. They now preserve the intended grouped ANSI decimal AVG fallback on Spark 4.x; the shared and Spark 3.x expectations are unchanged. The operator guide also explains that the guard applies when AVG still has a DECIMAL input after Spark optimization, including other functions in the same aggregate operator and queries without LIMIT. Validation passed all 129 plan checks on each of Spark 3.4.3, 3.5.9, 4.0.4, 4.1.3, and 4.2.0 with generation disabled (645 total). Current JVM classes and the actual loaded native library were verified. No implementation or native code changed; no new native build or performance result is claimed. The PR description has been updated, and the new hosted CI run is pending. -- 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]
