taiyang-li opened a new pull request, #12486: URL: https://github.com/apache/gluten/pull/12486
## What changes were proposed in this pull request? ColumnarPartialProject previously did not handle generic unmapped catalyst expressions shipped in user jars. When such a custom catalyst expression was not registered in `ExpressionMappings`, it could still reach native expression conversion and fail at runtime. This patch makes Velox detect those generic unmapped catalyst expressions earlier and route them through `ColumnarPartialProject` fallback. `ScalaUDF` and Hive UDF keep using their existing dedicated paths instead of being folded into the generic fallback path. ## How was this patch tested? - Added a Velox unit test with a custom `DummyUnmappedExpr` in `UDFPartialProjectSuite` - Ran `./dev/format-scala-code.sh` - Attempted `./build/mvn -pl backends-velox -Pbackends-velox -Pspark-3.5 -Pscala-2.12 -Dtest=none -Dsuites=org.apache.gluten.expression.UDFPartialProjectSuite test` - The Maven test build is currently blocked by pre-existing branch baseline compilation errors in files such as `FilterExecTransformer.scala` and `VeloxSparkPlanExecApi.scala`, which are unrelated to this patch ## Does this PR introduce _any_ user-facing change? No. ## Was this patch authored or co-authored using generative AI tooling? Yes. Generated-by: OpenAI GPT-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]
