rdblue commented on PR #9487: URL: https://github.com/apache/iceberg/pull/9487#issuecomment-1913713816
@jbonofre, I think @danielcweeks and I are both on the side of needing to have a single table. The less bad option is to have older clients fail to read metadata. There is no other solution because old clients don't know about additional tables. As I noted, old clients would ignore the views table and create conflicting tables. That leads to very ugly cases where tables and views conflict and you get invalid results. The remaining question is how to upgrade and from Dan's last comment, I think at least he and I are advocating for the same thing: it should be an explicit migration to add view support. I'm thinking that we should disable views if the column doesn't exist and have a catalog method to explicitly migrate to add view support. -- 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: issues-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
