xuanyuanking commented on pull request #30521: URL: https://github.com/apache/spark/pull/30521#issuecomment-737825126
Sorry for the misunderstanding. I thought you could agree with the current approach after adding comments/todo in code per https://github.com/apache/spark/pull/30521#issuecomment-737505823. Summarizing the current status: - For V1/V2 support, I think we already reach an agreement. I had created SPARK-33638 to track the V2 table support. I'll keep adding the discussion here into the Jira comments. - For the behavior of creating a table by default if not exists, this should be the only one left we need to get consensus. Personally, I prefer not to add a default param in the API if possible since it should be natural on the streaming side to follow the existing streaming sink behavior, e.g., Kafka creates a topic by default per https://github.com/apache/spark/pull/30521#issuecomment-737488404 and FileStreamSink creates a path per https://github.com/apache/spark/pull/30521#issuecomment-737308122. ---------------------------------------------------------------- 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]
