Repository: spark Updated Branches: refs/heads/branch-2.1 483624c2e -> d8548c8a7
[SPARK-18892][SQL] Alias percentile_approx approx_percentile ## What changes were proposed in this pull request? percentile_approx is the name used in Hive, and approx_percentile is the name used in Presto. approx_percentile is actually more consistent with our approx_count_distinct. Given the cost to alias SQL functions is low (one-liner), it'd be better to just alias them so it is easier to use. ## How was this patch tested? Technically I could add an end-to-end test to verify this one-line change, but it seemed too trivial to me. Author: Reynold Xin <r...@databricks.com> Closes #16300 from rxin/SPARK-18892. (cherry picked from commit 172a52f5d31337d90155feb7072381e8d5712288) Signed-off-by: Reynold Xin <r...@databricks.com> Project: http://git-wip-us.apache.org/repos/asf/spark/repo Commit: http://git-wip-us.apache.org/repos/asf/spark/commit/d8548c8a Tree: http://git-wip-us.apache.org/repos/asf/spark/tree/d8548c8a Diff: http://git-wip-us.apache.org/repos/asf/spark/diff/d8548c8a Branch: refs/heads/branch-2.1 Commit: d8548c8a7541bfa37761382edbb1892a145b2b71 Parents: 483624c Author: Reynold Xin <r...@databricks.com> Authored: Thu Dec 15 21:58:27 2016 -0800 Committer: Reynold Xin <r...@databricks.com> Committed: Thu Dec 15 21:58:33 2016 -0800 ---------------------------------------------------------------------- .../apache/spark/sql/catalyst/analysis/FunctionRegistry.scala | 1 + .../catalyst/expressions/aggregate/ApproximatePercentile.scala | 4 ++-- 2 files changed, 3 insertions(+), 2 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/spark/blob/d8548c8a/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/FunctionRegistry.scala ---------------------------------------------------------------------- diff --git a/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/FunctionRegistry.scala b/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/FunctionRegistry.scala index 2636afe..06b52a8 100644 --- a/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/FunctionRegistry.scala +++ b/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/FunctionRegistry.scala @@ -252,6 +252,7 @@ object FunctionRegistry { expression[Percentile]("percentile"), expression[Skewness]("skewness"), expression[ApproximatePercentile]("percentile_approx"), + expression[ApproximatePercentile]("approx_percentile"), expression[StddevSamp]("std"), expression[StddevSamp]("stddev"), expression[StddevPop]("stddev_pop"), http://git-wip-us.apache.org/repos/asf/spark/blob/d8548c8a/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/ApproximatePercentile.scala ---------------------------------------------------------------------- diff --git a/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/ApproximatePercentile.scala b/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/ApproximatePercentile.scala index 692cbd7..be9e610 100644 --- a/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/ApproximatePercentile.scala +++ b/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/ApproximatePercentile.scala @@ -61,9 +61,9 @@ import org.apache.spark.sql.types._ """, extended = """ Examples: - > SELECT percentile_approx(10.0, array(0.5, 0.4, 0.1), 100); + > SELECT _FUNC_(10.0, array(0.5, 0.4, 0.1), 100); [10.0,10.0,10.0] - > SELECT percentile_approx(10.0, 0.5, 100); + > SELECT _FUNC_(10.0, 0.5, 100); 10.0 """) case class ApproximatePercentile( --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@spark.apache.org For additional commands, e-mail: commits-h...@spark.apache.org