pvary commented on PR #10996: URL: https://github.com/apache/iceberg/pull/10996#issuecomment-2478468746
I expect that there will be users trying to use HiveCatalog with old Hive versions. Do we want to (could we) throw a meaningful exception when this happens? Maybe we can throw an exception in the `HiveVersion.calculalte()`? I am not sure about this, as this will cause an error to be thrown when Hive is on the classpath, even if it is not used. Is HiveVersion used anywhere after this change? -- 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
