HeartSaVioR commented on pull request #30521: URL: https://github.com/apache/spark/pull/30521#issuecomment-737871813
I think I have been answered lots of opinions, especially providing objection of the base assumption `table is a view of path` which leads the result that stream query should create table by default. Saying again, the assumption is only matched with file table, not DSv2 table. Revisit the @cloud-fan comment https://github.com/apache/spark/pull/30521#issuecomment-737038157 I can take the suggestion but with that I don't believe case 2 in @cloud-fan comment is rare. Either we should have two different methods like @cloud-fan proposed (`One possible solution is to add 2 methods insertTable and createAndInsertTable`), or add a flag param (no default value is preferred). We shouldn't restrict the behavior what users will be affected by. ---------------------------------------------------------------- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
