Github user cloud-fan commented on a diff in the pull request:

    https://github.com/apache/spark/pull/22043#discussion_r208884689
  
    --- Diff: 
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/basicLogicalOperators.scala
 ---
    @@ -367,6 +367,7 @@ case class AppendData(
           case (inAttr, outAttr) =>
               // names and types must match, nullability must be compatible
               inAttr.name == outAttr.name &&
    +          inAttr.resolved && outAttr.resolved &&
    --- End diff --
    
    I think it's more clear to write `table.resolved && query.resolved && 
query.output.size == table.output.size && ...`


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to