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

    https://github.com/apache/spark/pull/19327#discussion_r141934283
  
    --- Diff: 
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/UnsupportedOperationChecker.scala
 ---
    @@ -233,16 +234,54 @@ object UnsupportedOperationChecker {
                     throwError("Full outer joins with streaming 
DataFrames/Datasets are not supported")
                   }
     
    -            case LeftOuter | LeftSemi | LeftAnti =>
    +            case LeftSemi | LeftAnti =>
                   if (right.isStreaming) {
    -                throwError("Left outer/semi/anti joins with a streaming 
DataFrame/Dataset " +
    -                    "on the right is not supported")
    +                throwError("Left semi/anti joins with a streaming 
DataFrame/Dataset " +
    +                    "on the right are not supported")
                   }
     
    -            case RightOuter =>
    -              if (left.isStreaming) {
    -                throwError("Right outer join with a streaming 
DataFrame/Dataset on the left is " +
    -                    "not supported")
    +            // We support left and right outer streaming joins only in the 
stream+stream case.
    +            case LeftOuter | RightOuter =>
    +              if (joinType == LeftOuter && !left.isStreaming && 
right.isStreaming) {
    +                throwError("Left outer join with a streaming 
DataFrame/Dataset " +
    +                  "on the right and non-streaming on the left is not 
supported")
    --- End diff --
    
    ... and a static DataFrame/Dataset on the left is not supported


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to