[
https://issues.apache.org/jira/browse/FLINK-4565?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15804206#comment-15804206
]
ASF GitHub Bot commented on FLINK-4565:
---------------------------------------
Github user twalthr commented on a diff in the pull request:
https://github.com/apache/flink/pull/2870#discussion_r94803910
--- Diff:
flink-libraries/flink-table/src/main/scala/org/apache/flink/api/table/expressions/In.scala
---
@@ -0,0 +1,74 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements. See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership. The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License. You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package org.apache.flink.api.table.expressions
+
+import org.apache.calcite.rex.RexNode
+import org.apache.calcite.sql.SqlOperator
+import org.apache.calcite.sql.fun.SqlStdOperatorTable
+import org.apache.calcite.tools.RelBuilder
+import org.apache.flink.api.common.typeinfo.BasicTypeInfo._
+import org.apache.flink.api.common.typeinfo.TypeInformation
+import org.apache.flink.api.table.typeutils.TypeCheckUtils._
+import org.apache.flink.api.table.validate.{ValidationResult,
ValidationFailure, ValidationSuccess}
+
+case class In(expression: Expression, subquery: Seq[Expression]) extends
Expression {
+
+ def this(expressions: Seq[Expression]) = this(expressions.head,
expressions.tail)
+
+ override def toString = s"$expression.in(${subquery.mkString(", ")})"
+
+ /**
+ * List of child nodes that should be considered when doing
transformations. Other values
+ * in the Product will not be transformed, only handed through.
+ */
+ override private[flink] def children: Seq[Expression] = expression +:
subquery.distinct
+
+ override private[flink] def toRexNode(implicit relBuilder: RelBuilder):
RexNode = {
+ relBuilder.call(SqlStdOperatorTable.IN, children.map(_.toRexNode): _*)
+ }
+
+ override private[flink] def validateInput(): ValidationResult = {
+ if (children.tail.contains(null)) {
+ ValidationFailure("Operands on right side of IN operator must be not
null")
+ } else {
+ val types = children.tail.map(_.resultType)
+ if (types.distinct.length != 1) {
+ ValidationFailure(
+ s"Types on the right side of IN operator must be the same, got
${types.mkString(", ")}."
+ )
+ } else {
+ (children.head.resultType, children.tail.head.resultType) match {
+ case (lType, rType) if isNumeric(lType) && isNumeric(rType) =>
ValidationSuccess
+ case (lType, rType) if isComparable(lType) && lType == rType =>
ValidationSuccess
+ case (lType, rType) =>
+ ValidationFailure(
+ s"Types on the both side of IN operator must be the same,
got $lType and $rType"
--- End diff --
"Types on both sides of the IN operator must be numeric or of the same
comparable type"
> Support for SQL IN operator
> ---------------------------
>
> Key: FLINK-4565
> URL: https://issues.apache.org/jira/browse/FLINK-4565
> Project: Flink
> Issue Type: Improvement
> Components: Table API & SQL
> Reporter: Timo Walther
> Assignee: Nikolay Vasilishin
>
> It seems that Flink SQL supports the uncorrelated sub-query IN operator. But
> it should also be available in the Table API and tested.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
