xianyinxin opened a new pull request #26464: [SPARK-29835] Remove the 
unnecessary conversion from Statement to LogicalPlan for D…
URL: https://github.com/apache/spark/pull/26464
 
 
   ### What changes were proposed in this pull request?
   
   The current parse and analyze flow for DELETE is: 1, the SQL string will be 
firstly parsed to `DeleteFromStatement`; 2, the `DeleteFromStatement` be 
converted to `DeleteFromTable`. However, the SQL string can be parsed to 
`DeleteFromTable` directly, where a `DeleteFromStatement` seems to be redundant.
   
   It is the same for UPDATE.
   
   This pr removes the unnecessary `DeleteFromStatement` and 
`UpdateTableStatement`.
   
   ### Why are the changes needed?
   
   This makes the codes for DELETE and UPDATE cleaner, and keep align with 
MERGE INTO.
   
   ### Does this PR introduce any user-facing change?
   No.
   
   ### How was this patch tested?
   Existed tests and new tests.
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

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

Reply via email to