aboutsummaryrefslogtreecommitdiff
path: root/sql/core
diff options
context:
space:
mode:
author云峤 <chensong.cs@alibaba-inc.com>2015-05-05 13:23:53 -0700
committerReynold Xin <rxin@databricks.com>2015-05-05 13:23:53 -0700
commit735bc3d042b1e3e12de57b66f166af14254ad314 (patch)
tree853b8c985baf71c78ab51ddc7724dfad630523c6 /sql/core
parent489700c809a7c0a836538f3d0bd58bed609e8768 (diff)
downloadspark-735bc3d042b1e3e12de57b66f166af14254ad314.tar.gz
spark-735bc3d042b1e3e12de57b66f166af14254ad314.tar.bz2
spark-735bc3d042b1e3e12de57b66f166af14254ad314.zip
[SPARK-7294][SQL] ADD BETWEEN
Author: 云峤 <chensong.cs@alibaba-inc.com> Author: kaka1992 <kaka_1992@163.com> Closes #5839 from kaka1992/master and squashes the following commits: b15360d [kaka1992] Fix python unit test in sql/test. =_= I forget to commit this file last time. f928816 [kaka1992] Fix python style in sql/test. d2e7f72 [kaka1992] Fix python style in sql/test. c54d904 [kaka1992] Fix empty map bug. 7e64d1e [云峤] Update 7b9b858 [云峤] undo f080f8d [云峤] update pep8 76f0c51 [云峤] Merge remote-tracking branch 'remotes/upstream/master' 7d62368 [云峤] [SPARK-7294] ADD BETWEEN baf839b [云峤] [SPARK-7294] ADD BETWEEN d11d5b9 [云峤] [SPARK-7294] ADD BETWEEN
Diffstat (limited to 'sql/core')
-rw-r--r--sql/core/src/main/scala/org/apache/spark/sql/Column.scala9
-rw-r--r--sql/core/src/test/scala/org/apache/spark/sql/ColumnExpressionSuite.scala14
2 files changed, 23 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 33f9d0b37d..c0503bf047 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
@@ -296,6 +296,15 @@ class Column(protected[sql] val expr: Expression) extends Logging {
def eqNullSafe(other: Any): Column = this <=> other
/**
+ * True if the current column is between the lower bound and upper bound, inclusive.
+ *
+ * @group java_expr_ops
+ */
+ def between(lowerBound: Any, upperBound: Any): Column = {
+ (this >= lowerBound) && (this <= upperBound)
+ }
+
+ /**
* True if the current expression is null.
*
* @group expr_ops
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 6322faf4d9..3c1ad656fc 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
@@ -208,6 +208,20 @@ class ColumnExpressionSuite extends QueryTest {
testData2.collect().toSeq.filter(r => r.getInt(0) <= r.getInt(1)))
}
+ test("between") {
+ val testData = TestSQLContext.sparkContext.parallelize(
+ (0, 1, 2) ::
+ (1, 2, 3) ::
+ (2, 1, 0) ::
+ (2, 2, 4) ::
+ (3, 1, 6) ::
+ (3, 2, 0) :: Nil).toDF("a", "b", "c")
+ val expectAnswer = testData.collect().toSeq.
+ filter(r => r.getInt(0) >= r.getInt(1) && r.getInt(0) <= r.getInt(2))
+
+ checkAnswer(testData.filter($"a".between($"b", $"c")), expectAnswer)
+ }
+
val booleanData = TestSQLContext.createDataFrame(TestSQLContext.sparkContext.parallelize(
Row(false, false) ::
Row(false, true) ::