naan31 commented on PR #56138:
URL: https://github.com/apache/spark/pull/56138#issuecomment-4552849628
@srielau - I've added a regression test for issue #55392 as you requested.
**What the test does:**
The test verifies that named parameters like `:x` work correctly in
`spark.sql()`
when provided in the args dict, WITHOUT needing the legacy parameter
substitution config.
**Test cases:**
1. Single parameter: `spark.sql("SELECT :x", args=Map("x" -> 1))` returns
`Row(1)`
2. Multiple parameters: `spark.sql("SELECT :x, :y", args=Map("x" -> 1, "y"
-> 2))` returns `Row(1, 2)`
**Why this matters:**
This reproduces the exact issue from #55392 where PySpark 4.1.1 incorrectly
rejected
these named parameters even though they were provided in the args.
**Environment:**
- OS: macOS
- Java version: openjdk version "17.0.17" 2025-10-21
- Scala: 2.13.x
The test is now ready for the CI/CD pipeline to validate. If it passes, it
proves
the regression exists and your fix resolves it. If it fails, it will help us
understand why the issue cannot be reproduced.
--
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]
