akshaytayal opened a new pull request, #13431:
URL: https://github.com/apache/gluten/pull/13431

   ## What changes are proposed in this pull request?
   
   Re-enables two Spark 4.2 tests that #13163 disabled:
   
   - `GlutenBroadcastExchangeSuite`: `SPARK-56455: broadcast should fail when 
table size exceeds maxBroadcastTableSize`. Gluten's columnar broadcast already 
throws Spark's `_LEGACY_ERROR_TEMP_2249` error for a table over 
`spark.sql.maxBroadcastTableSize` (#13192), so the test passes unchanged.
   - `GlutenReplaceHashWithSortAggSuite`: `Gluten - do not replace hash 
aggregate if child does not have sort order`. Since SPARK-55551, 
`BroadcastHashJoinExec` also expands the streamed side's range partitioning to 
the build keys, so the final aggregate needs no shuffle and Gluten merges the 
partial and final aggregates into one complete-mode aggregate. The expected 
aggregate count is updated from 2 to 1; query results are unchanged.
   
   This is a test-only change. Part of #13178 (umbrella #13181).
   
   ## How was this patch tested?
   
   Locally on Spark 4.2: `GlutenBroadcastExchangeSuite` and 
`GlutenReplaceHashWithSortAggSuite` pass, including both re-enabled tests. 
Spark 4.2 UT CI.
   
   ## Was this patch authored or co-authored using generative AI tooling?
   
   Generated-by: GitHub Copilot CLI (Claude Opus 5.5)


-- 
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]

Reply via email to