[ 
https://issues.apache.org/jira/browse/FLINK-7853?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16208450#comment-16208450
 ] 

ASF GitHub Bot commented on FLINK-7853:
---------------------------------------

Github user fhueske commented on a diff in the pull request:

    https://github.com/apache/flink/pull/4842#discussion_r145257154
  
    --- Diff: 
flink-libraries/flink-table/src/main/scala/org/apache/flink/table/plan/logical/operators.scala
 ---
    @@ -476,20 +481,30 @@ case class Join(
             } else {
               nonEquiJoinPredicateFound = true
             }
    +      // The boolean literal should be valid condition type.
    +      case x: Literal if x.resultType == Types.BOOLEAN =>
           case x => failValidation(
             s"Unsupported condition type: ${x.getClass.getSimpleName}. 
Condition: $x")
         }
     
         validateConditions(expression, isAndBranch = true)
    -    if (!equiJoinPredicateFound) {
    -      failValidation(
    -        s"Invalid join condition: $expression. At least one equi-join 
predicate is " +
    -          s"required.")
    -    }
    -    if (joinType != JoinType.INNER && (nonEquiJoinPredicateFound || 
localPredicateFound)) {
    -      failValidation(
    -        s"Invalid join condition: $expression. Non-equality join 
predicates or local" +
    -          s" predicates are not supported in outer joins.")
    +
    +    // Due to CALCITE-2004, we cannot accept join predicates except 
literal true for TableFunction
    +    // left outer join.
    +    if (correlated && right.isInstanceOf[LogicalTableFunctionCall] && 
joinType != JoinType.INNER ) {
    +      if (!alwaysTrue) failValidation("TableFunction left outer join 
predicates can only be " +
    --- End diff --
    
    `predicates` -> `predicate`


> Reject table function outer joins with predicates in Table API
> --------------------------------------------------------------
>
>                 Key: FLINK-7853
>                 URL: https://issues.apache.org/jira/browse/FLINK-7853
>             Project: Flink
>          Issue Type: Sub-task
>          Components: Table API & SQL
>            Reporter: Xingcan Cui
>            Assignee: Xingcan Cui
>            Priority: Blocker
>             Fix For: 1.4.0
>
>
> Due to CALCITE-2004, the table function outer joins can not be normally 
> executed. We should cover it up by rejecting join predicates temporarily, 
> until the issue is fixed in Calcite.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to