Revision 80c44329fec47f7eaa0f25b3dff96e2ee06c80c9 authored by wforget on 30 July 2024, 03:25:39 UTC, committed by Wenchen Fan on 30 July 2024, 03:25:39 UTC
### What changes were proposed in this pull request? Add `failOnError` argument for `UrlDecode` to replace `TryEval` in `TryUrlDecode`. ### Why are the changes needed? Address https://github.com/apache/spark/pull/47294#discussion_r1681150787 > I'm not a big fan of TryEval as it catches all the exceptions, including the ones not from UrlDecode, but from its input expressions. > > We should add a boolean flag in UrlDecode to control the null-on-error behavior. ### Does this PR introduce _any_ user-facing change? No ### How was this patch tested? existing unit tests ### Was this patch authored or co-authored using generative AI tooling? No Closes #47514 from wForget/try_url_decode. Authored-by: wforget <643348094@qq.com> Signed-off-by: Wenchen Fan <wenchen@databricks.com>
1 parent 308669f
Computing file changes ...
