acvictor opened a new pull request, #11757:
URL: https://github.com/apache/gluten/pull/11757
## What changes are proposed in this pull request?
This PR replaces `None` on the default case, so getOrElse correctly falls
through.
Currently, `tryTransformWithoutExpressionMapping` returns `Some(null)` for
unmatched expressions instead of `None` and the caller does `.getOrElse {
transformExpression(...) }`. Since Some(null).getOrElse(fallback) returns null
without invoking the fallback, some paths can silently get a null transformer
which will lead to a downstream NPE.
## How was this patch tested?
Existing UTs.
## Was this patch authored or co-authored using generative AI tooling?
No
--
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]
