massdosage commented on pull request #1505: URL: https://github.com/apache/iceberg/pull/1505#issuecomment-698445928
> > The changes look good. Just a quick sanity check: what happens in Hive if the serde or storage handler class is missing? Will this break metastore operations on the table? > > I remember that a missing class used to break any query that loaded the table, including DROP TABLE. So you couldn't even clean up problems. > > That was my fear as well, but did not have time to test them out yet. It might even cause issues for users who are not using the Hive tables, just want to use HiveCatalog to store the table versions. I'm pretty sure that if you try to perform any operations on tables with the StorageHandler set on them, and you don't have that class on the classpath, it will fail with an error. I've been in exactly the situation you describe where I couldn't issue a DROP TABLE for a table that was created by a test storage handler that no longer existed. ---------------------------------------------------------------- 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]
