From d6dc603ed480abaa1e801cf626cd3b2153d101fc Mon Sep 17 00:00:00 2001
From: gatorsmile <gatorsmile@gmail.com>
Date: Tue, 7 Feb 2017 09:59:16 +0800
Subject: [PATCH] [SPARK-19441][SQL] Remove IN type coercion from
PromoteStrings
### What changes were proposed in this pull request?
The removed codes for `IN` are not reachable, because the previous rule `InConversion` already resolves the type coercion issues.
### How was this patch tested?
N/A
Author: gatorsmile <gatorsmile@gmail.com>
Closes #16783 from gatorsmile/typeCoercionIn.
---
.../spark/sql/catalyst/analysis/TypeCoercion.scala | 9 ---------
1 file changed, 9 deletions(-)
diff --git a/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/TypeCoercion.scala b/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/TypeCoercion.scala
index c6242e01ee..b636c31703 100644
--- a/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/TypeCoercion.scala
+++ b/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/TypeCoercion.scala
@@ -335,15 +335,6 @@ object TypeCoercion {
if left.dataType != StringType =>
p.makeCopy(Array(left, Cast(right, left.dataType)))
- case i @ In(a @ DateType(), b) if b.forall(_.dataType == StringType) =>
- i.makeCopy(Array(Cast(a, StringType), b))
- case i @ In(a @ TimestampType(), b) if b.forall(_.dataType == StringType) =>
- i.makeCopy(Array(a, b.map(Cast(_, TimestampType))))
- case i @ In(a @ DateType(), b) if b.forall(_.dataType == TimestampType) =>
- i.makeCopy(Array(Cast(a, StringType), b.map(Cast(_, StringType))))
- case i @ In(a @ TimestampType(), b) if b.forall(_.dataType == DateType) =>
- i.makeCopy(Array(Cast(a, StringType), b.map(Cast(_, StringType))))
-
case Sum(e @ StringType()) => Sum(Cast(e, DoubleType))
case Average(e @ StringType()) => Average(Cast(e, DoubleType))
case StddevPop(e @ StringType()) => StddevPop(Cast(e, DoubleType))
--
GitLab