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
| File | Mode | Size |
|---|---|---|
| .github | ||
| R | ||
| assembly | ||
| bin | ||
| binder | ||
| build | ||
| common | ||
| conf | ||
| connect | ||
| connector | ||
| core | ||
| data | ||
| dev | ||
| docs | ||
| examples | ||
| graphx | ||
| hadoop-cloud | ||
| launcher | ||
| licenses | ||
| licenses-binary | ||
| mllib | ||
| mllib-local | ||
| project | ||
| python | ||
| repl | ||
| resource-managers | ||
| sbin | ||
| sql | ||
| streaming | ||
| tools | ||
| ui-test | ||
| .asf.yaml | -rw-r--r-- | 1.3 KB |
| .gitattributes | -rw-r--r-- | 130 bytes |
| .gitignore | -rw-r--r-- | 1.9 KB |
| CONTRIBUTING.md | -rw-r--r-- | 997 bytes |
| LICENSE | -rw-r--r-- | 13.2 KB |
| LICENSE-binary | -rw-r--r-- | 21.7 KB |
| NOTICE | -rw-r--r-- | 2.0 KB |
| NOTICE-binary | -rw-r--r-- | 51.6 KB |
| README.md | -rw-r--r-- | 4.3 KB |
| pom.xml | -rw-r--r-- | 140.3 KB |
| scalastyle-config.xml | -rw-r--r-- | 25.9 KB |
Computing file changes ...
