[GitHub] [spark] ulysses-you commented on a change in pull request #31372: [SPARK-34272][SQL] Pretty SQL should check NonSQLExpression

2021-01-29 Thread GitBox
ulysses-you commented on a change in pull request #31372: URL: https://github.com/apache/spark/pull/31372#discussion_r566647200 ## File path: sql/core/src/test/resources/sql-tests/results/udf/postgreSQL/udf-aggregates_part1.sql.out ## @@ -109,7 +109,7 @@ struct -- !query SEL

[GitHub] [spark] ulysses-you commented on a change in pull request #31372: [SPARK-34272][SQL] Pretty SQL should check NonSQLExpression

2021-01-27 Thread GitBox
ulysses-you commented on a change in pull request #31372: URL: https://github.com/apache/spark/pull/31372#discussion_r565825593 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/package.scala ## @@ -125,20 +125,34 @@ package object util extends Loggi

[GitHub] [spark] ulysses-you commented on a change in pull request #31372: [SPARK-34272][SQL] Pretty SQL should check NonSQLExpression

2021-01-27 Thread GitBox
ulysses-you commented on a change in pull request #31372: URL: https://github.com/apache/spark/pull/31372#discussion_r565787380 ## File path: sql/core/src/test/resources/sql-tests/results/udf/postgreSQL/udf-aggregates_part1.sql.out ## @@ -109,7 +109,7 @@ struct -- !query SEL