SinghAsDev commented on pull request #4018: URL: https://github.com/apache/iceberg/pull/4018#issuecomment-1030073667
Thanks for the thoughts @rdblue . I think what you said about iceberg not having a concept of 'external' is definitely a good point. However, Iceberg's spark catalog does allow spark sql users to create iceberg tables from spark sql, and spark sql supports the 'external' concept. This leads to the situation where unexpected dropping of data happen. I think we can maybe make not supporting of external tables explicit by throwing a informative exception during create tables in spark catalog. @rdblue would that be a reasonable approach? -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
