aboutsummaryrefslogtreecommitdiff
path: root/sql/core
diff options
context:
space:
mode:
authorReynold Xin <rxin@databricks.com>2015-05-12 21:43:34 -0700
committerReynold Xin <rxin@databricks.com>2015-05-12 21:43:34 -0700
commit97dee313f23b00f15638cb72a4a80c1f197f8a9d (patch)
treeb6718297822929afef06cb8550e765c8ad637efe /sql/core
parent8fd55358b7fc1c7545d823bef7b39769f731c1ee (diff)
downloadspark-97dee313f23b00f15638cb72a4a80c1f197f8a9d.tar.gz
spark-97dee313f23b00f15638cb72a4a80c1f197f8a9d.tar.bz2
spark-97dee313f23b00f15638cb72a4a80c1f197f8a9d.zip
[SPARK-7321][SQL] Add Column expression for conditional statements (when/otherwise)
This builds on https://github.com/apache/spark/pull/5932 and should close https://github.com/apache/spark/pull/5932 as well. As an example: ```python df.select(when(df['age'] == 2, 3).otherwise(4).alias("age")).collect() ``` Author: Reynold Xin <rxin@databricks.com> Author: kaka1992 <kaka_1992@163.com> Closes #6072 from rxin/when-expr and squashes the following commits: 8f49201 [Reynold Xin] Throw exception if otherwise is applied twice. 0455eda [Reynold Xin] Reset run-tests. bfb9d9f [Reynold Xin] Updated documentation and test cases. 762f6a5 [Reynold Xin] Merge pull request #5932 from kaka1992/IFCASE 95724c6 [kaka1992] Update 8218d0a [kaka1992] Update 801009e [kaka1992] Update 76d6346 [kaka1992] [SPARK-7321][SQL] Add Column expression for conditional statements (if, case)
Diffstat (limited to 'sql/core')
-rw-r--r--sql/core/src/main/scala/org/apache/spark/sql/Column.scala61
-rw-r--r--sql/core/src/main/scala/org/apache/spark/sql/functions.scala24
-rw-r--r--sql/core/src/test/scala/org/apache/spark/sql/ColumnExpressionSuite.scala21
3 files changed, 106 insertions, 0 deletions
diff --git a/sql/core/src/main/scala/org/apache/spark/sql/Column.scala b/sql/core/src/main/scala/org/apache/spark/sql/Column.scala
index 4773dedf72..42f5bcda49 100644
--- a/sql/core/src/main/scala/org/apache/spark/sql/Column.scala
+++ b/sql/core/src/main/scala/org/apache/spark/sql/Column.scala
@@ -328,6 +328,67 @@ class Column(protected[sql] val expr: Expression) extends Logging {
def eqNullSafe(other: Any): Column = this <=> other
/**
+ * Evaluates a list of conditions and returns one of multiple possible result expressions.
+ * If otherwise is not defined at the end, null is returned for unmatched conditions.
+ *
+ * {{{
+ * // Example: encoding gender string column into integer.
+ *
+ * // Scala:
+ * people.select(when(people("gender") === "male", 0)
+ * .when(people("gender") === "female", 1)
+ * .otherwise(2))
+ *
+ * // Java:
+ * people.select(when(col("gender").equalTo("male"), 0)
+ * .when(col("gender").equalTo("female"), 1)
+ * .otherwise(2))
+ * }}}
+ *
+ * @group expr_ops
+ */
+ def when(condition: Column, value: Any):Column = this.expr match {
+ case CaseWhen(branches: Seq[Expression]) =>
+ CaseWhen(branches ++ Seq(lit(condition).expr, lit(value).expr))
+ case _ =>
+ throw new IllegalArgumentException(
+ "when() can only be applied on a Column previously generated by when() function")
+ }
+
+ /**
+ * Evaluates a list of conditions and returns one of multiple possible result expressions.
+ * If otherwise is not defined at the end, null is returned for unmatched conditions.
+ *
+ * {{{
+ * // Example: encoding gender string column into integer.
+ *
+ * // Scala:
+ * people.select(when(people("gender") === "male", 0)
+ * .when(people("gender") === "female", 1)
+ * .otherwise(2))
+ *
+ * // Java:
+ * people.select(when(col("gender").equalTo("male"), 0)
+ * .when(col("gender").equalTo("female"), 1)
+ * .otherwise(2))
+ * }}}
+ *
+ * @group expr_ops
+ */
+ def otherwise(value: Any):Column = this.expr match {
+ case CaseWhen(branches: Seq[Expression]) =>
+ if (branches.size % 2 == 0) {
+ CaseWhen(branches :+ lit(value).expr)
+ } else {
+ throw new IllegalArgumentException(
+ "otherwise() can only be applied once on a Column previously generated by when()")
+ }
+ case _ =>
+ throw new IllegalArgumentException(
+ "otherwise() can only be applied on a Column previously generated by when()")
+ }
+
+ /**
* True if the current column is between the lower bound and upper bound, inclusive.
*
* @group java_expr_ops
diff --git a/sql/core/src/main/scala/org/apache/spark/sql/functions.scala b/sql/core/src/main/scala/org/apache/spark/sql/functions.scala
index 215787e40b..099e1d8f03 100644
--- a/sql/core/src/main/scala/org/apache/spark/sql/functions.scala
+++ b/sql/core/src/main/scala/org/apache/spark/sql/functions.scala
@@ -420,6 +420,30 @@ object functions {
def not(e: Column): Column = !e
/**
+ * Evaluates a list of conditions and returns one of multiple possible result expressions.
+ * If otherwise is not defined at the end, null is returned for unmatched conditions.
+ *
+ * {{{
+ * // Example: encoding gender string column into integer.
+ *
+ * // Scala:
+ * people.select(when(people("gender") === "male", 0)
+ * .when(people("gender") === "female", 1)
+ * .otherwise(2))
+ *
+ * // Java:
+ * people.select(when(col("gender").equalTo("male"), 0)
+ * .when(col("gender").equalTo("female"), 1)
+ * .otherwise(2))
+ * }}}
+ *
+ * @group normal_funcs
+ */
+ def when(condition: Column, value: Any): Column = {
+ CaseWhen(Seq(condition.expr, lit(value).expr))
+ }
+
+ /**
* Generate a random column with i.i.d. samples from U[0.0, 1.0].
*
* @group normal_funcs
diff --git a/sql/core/src/test/scala/org/apache/spark/sql/ColumnExpressionSuite.scala b/sql/core/src/test/scala/org/apache/spark/sql/ColumnExpressionSuite.scala
index d96186c268..269e185543 100644
--- a/sql/core/src/test/scala/org/apache/spark/sql/ColumnExpressionSuite.scala
+++ b/sql/core/src/test/scala/org/apache/spark/sql/ColumnExpressionSuite.scala
@@ -255,6 +255,27 @@ class ColumnExpressionSuite extends QueryTest {
Row(false, true) :: Row(true, false) :: Row(true, true) :: Nil)
}
+ test("SPARK-7321 when conditional statements") {
+ val testData = (1 to 3).map(i => (i, i.toString)).toDF("key", "value")
+
+ checkAnswer(
+ testData.select(when($"key" === 1, -1).when($"key" === 2, -2).otherwise(0)),
+ Seq(Row(-1), Row(-2), Row(0))
+ )
+
+ // Without the ending otherwise, return null for unmatched conditions.
+ // Also test putting a non-literal value in the expression.
+ checkAnswer(
+ testData.select(when($"key" === 1, lit(0) - $"key").when($"key" === 2, -2)),
+ Seq(Row(-1), Row(-2), Row(null))
+ )
+
+ // Test error handling for invalid expressions.
+ intercept[IllegalArgumentException] { $"key".when($"key" === 1, -1) }
+ intercept[IllegalArgumentException] { $"key".otherwise(-1) }
+ intercept[IllegalArgumentException] { when($"key" === 1, -1).otherwise(-1).otherwise(-1) }
+ }
+
test("sqrt") {
checkAnswer(
testData.select(sqrt('key)).orderBy('key.asc),