aboutsummaryrefslogtreecommitdiff
path: root/sql/catalyst/src
diff options
context:
space:
mode:
authorReynold Xin <rxin@databricks.com>2016-12-15 21:58:27 -0800
committerReynold Xin <rxin@databricks.com>2016-12-15 21:58:27 -0800
commit172a52f5d31337d90155feb7072381e8d5712288 (patch)
tree635a2fa4792f9974fd6e874985f68aba8d6cbbde /sql/catalyst/src
parent5a44f18a2a114bdd37b6714d81f88cb68148f0c9 (diff)
downloadspark-172a52f5d31337d90155feb7072381e8d5712288.tar.gz
spark-172a52f5d31337d90155feb7072381e8d5712288.tar.bz2
spark-172a52f5d31337d90155feb7072381e8d5712288.zip
[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 <rxin@databricks.com> Closes #16300 from rxin/SPARK-18892.
Diffstat (limited to 'sql/catalyst/src')
-rw-r--r--sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/FunctionRegistry.scala1
-rw-r--r--sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/ApproximatePercentile.scala4
2 files changed, 3 insertions, 2 deletions
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 5d065d736e..2b214c3c9d 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"),
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 c2cd8951bc..01792aef37 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(