cloud-fan commented on issue #24798: [SPARK-27724][SQL] Implement REPLACE TABLE and REPLACE TABLE AS SELECT with V2 URL: https://github.com/apache/spark/pull/24798#issuecomment-511271344 I was not saying that REPLACE TABLE isn't important, I just want to emphasize that, it's better to have smaller Spark PRs when possible. Many people try to enforce it when reviewing PRs, and I don't want to break the rule here. It not only easy the review, but also make the commit history clearer. This PR does 2 things: 1. introduce `StagingTableCatalog` API 2. implement REPLACE TABLE (depends on 1) We can create a new PR for `StagingTableCatalog`, and keep this PR for review/discussion. After the new PR is merged, rebase this PR. This is also what I do for individual stuff in big PRs: https://github.com/apache/spark/pull/25107
---------------------------------------------------------------- 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]
