[GitHub] [spark] yaooqinn edited a comment on pull request #28106: [SPARK-31335][SQL] Add try function support

2020-05-08 Thread GitBox
yaooqinn edited a comment on pull request #28106: URL: https://github.com/apache/spark/pull/28106#issuecomment-625671160 One more thing, the scope that the `try` can cover is not that large you have imagined. The cases you listed is actually are all invalid, e.g. sum(a, b)??? ```sql

[GitHub] [spark] yaooqinn edited a comment on pull request #28106: [SPARK-31335][SQL] Add try function support

2020-05-07 Thread GitBox
yaooqinn edited a comment on pull request #28106: URL: https://github.com/apache/spark/pull/28106#issuecomment-625648533 I see the concerns. The behavior of `try` here is simply that it wraps a `try-catch` for its root child expression. And the meanings of `try` will be spread based