You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@spark.apache.org by rx...@apache.org on 2015/05/12 19:28:46 UTC
spark git commit: [DataFrame][minor] cleanup unapply methods in
DataTypes
Repository: spark
Updated Branches:
refs/heads/master d86ce8458 -> 831504cf6
[DataFrame][minor] cleanup unapply methods in DataTypes
Author: Wenchen Fan <cl...@outlook.com>
Closes #6079 from cloud-fan/unapply and squashes the following commits:
40da442 [Wenchen Fan] one more
7d90a05 [Wenchen Fan] cleanup unapply in DataTypes
Project: http://git-wip-us.apache.org/repos/asf/spark/repo
Commit: http://git-wip-us.apache.org/repos/asf/spark/commit/831504cf
Tree: http://git-wip-us.apache.org/repos/asf/spark/tree/831504cf
Diff: http://git-wip-us.apache.org/repos/asf/spark/diff/831504cf
Branch: refs/heads/master
Commit: 831504cf6bde6b1131005d5552e56a842725c84c
Parents: d86ce84
Author: Wenchen Fan <cl...@outlook.com>
Authored: Tue May 12 10:28:40 2015 -0700
Committer: Reynold Xin <rx...@databricks.com>
Committed: Tue May 12 10:28:40 2015 -0700
----------------------------------------------------------------------
.../scala/org/apache/spark/sql/types/DataType.scala | 15 +++------------
1 file changed, 3 insertions(+), 12 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/spark/blob/831504cf/sql/catalyst/src/main/scala/org/apache/spark/sql/types/DataType.scala
----------------------------------------------------------------------
diff --git a/sql/catalyst/src/main/scala/org/apache/spark/sql/types/DataType.scala b/sql/catalyst/src/main/scala/org/apache/spark/sql/types/DataType.scala
index 0992a7c..a0b2616 100644
--- a/sql/catalyst/src/main/scala/org/apache/spark/sql/types/DataType.scala
+++ b/sql/catalyst/src/main/scala/org/apache/spark/sql/types/DataType.scala
@@ -47,10 +47,7 @@ abstract class DataType {
* ...
* }}}
*/
- private[sql] def unapply(a: Expression): Boolean = a match {
- case e: Expression if e.dataType == this => true
- case _ => false
- }
+ private[sql] def unapply(e: Expression): Boolean = e.dataType == this
/**
* The default size of a value of this data type, used internally for size estimation.
@@ -137,10 +134,7 @@ private[sql] object IntegralType {
* ...
* }}}
*/
- def unapply(a: Expression): Boolean = a match {
- case e: Expression if e.dataType.isInstanceOf[IntegralType] => true
- case _ => false
- }
+ def unapply(e: Expression): Boolean = e.dataType.isInstanceOf[IntegralType]
}
@@ -157,10 +151,7 @@ private[sql] object FractionalType {
* ...
* }}}
*/
- def unapply(a: Expression): Boolean = a match {
- case e: Expression if e.dataType.isInstanceOf[FractionalType] => true
- case _ => false
- }
+ def unapply(e: Expression): Boolean = e.dataType.isInstanceOf[FractionalType]
}
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@spark.apache.org
For additional commands, e-mail: commits-help@spark.apache.org