[GitHub] spark pull request #21187: [SPARK-24035][SQL] SQL syntax for Pivot

GitHub user maryannxue opened a pull request: https://github.com/apache/spark/pull/21187 [SPARK-24035][SQL] SQL syntax for Pivot ## What changes were proposed in this pull request? Add SQL support for Pivot according to Pivot grammar defined by Oracle (https://docs.oracle.c

[GitHub] spark pull request #21187: [SPARK-24035][SQL] SQL syntax for Pivot

Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/21187#discussion_r185084802 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/PivotSuite.scala --- @@ -0,0 +1,197 @@ +/* + * Licensed to the Apache Software Foundation (ASF) u

[GitHub] spark pull request #21187: [SPARK-24035][SQL] SQL syntax for Pivot

Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/21187#discussion_r185258526 --- Diff: sql/catalyst/src/main/antlr4/org/apache/spark/sql/catalyst/parser/SqlBase.g4 --- @@ -805,6 +810,7 @@ RIGHT: 'RIGHT'; FULL: 'FULL'; NA

[GitHub] spark pull request #21187: [SPARK-24035][SQL] SQL syntax for Pivot

Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/21187#discussion_r185260075 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/basicLogicalOperators.scala --- @@ -691,7 +691,9 @@ case class Pivot(

[GitHub] spark pull request #21187: [SPARK-24035][SQL] SQL syntax for Pivot

Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/21187#discussion_r185259275 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/basicLogicalOperators.scala --- @@ -691,7 +691,9 @@ case class Pivot(

[GitHub] spark pull request #21187: [SPARK-24035][SQL] SQL syntax for Pivot

Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/21187#discussion_r185264657 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/basicLogicalOperators.scala --- @@ -691,7 +691,9 @@ case class Pivot(

[GitHub] spark pull request #21187: [SPARK-24035][SQL] SQL syntax for Pivot

Github user maryannxue commented on a diff in the pull request: https://github.com/apache/spark/pull/21187#discussion_r185273161 --- Diff: sql/catalyst/src/main/antlr4/org/apache/spark/sql/catalyst/parser/SqlBase.g4 --- @@ -805,6 +810,7 @@ RIGHT: 'RIGHT'; FULL: 'FULL'; NA

[GitHub] spark pull request #21187: [SPARK-24035][SQL] SQL syntax for Pivot

Github user maryannxue commented on a diff in the pull request: https://github.com/apache/spark/pull/21187#discussion_r185278179 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/basicLogicalOperators.scala --- @@ -691,7 +691,9 @@ case class Pivot(

[GitHub] spark pull request #21187: [SPARK-24035][SQL] SQL syntax for Pivot

Github user maryannxue commented on a diff in the pull request: https://github.com/apache/spark/pull/21187#discussion_r185278754 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/basicLogicalOperators.scala --- @@ -691,7 +691,9 @@ case class Pivot(

[GitHub] spark pull request #21187: [SPARK-24035][SQL] SQL syntax for Pivot

Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/21187#discussion_r185641299 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/basicLogicalOperators.scala --- @@ -686,13 +686,26 @@ case class Group

[GitHub] spark pull request #21187: [SPARK-24035][SQL] SQL syntax for Pivot

Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/21187 --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org