Github user jose-torres commented on a diff in the pull request:

    https://github.com/apache/spark/pull/21305#discussion_r190695611
  
    --- Diff: 
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/basicLogicalOperators.scala
 ---
    @@ -344,6 +344,36 @@ case class Join(
       }
     }
     
    +/**
    + * Append data to an existing DataSourceV2 table.
    + */
    +case class AppendData(
    +    table: LogicalPlan,
    --- End diff --
    
    Do we still want to represent the table as a generic LogicalPlan rather 
than a separate table type? (Maybe the answer's yes; I'm not really clear on 
why this happens in the current code.)


---

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

Reply via email to