huaxingao commented on PR #10766: URL: https://github.com/apache/iceberg/pull/10766#issuecomment-2247098069
When constructing a new `AnalysisException`, if we can find a corresponding error class in Spark, we should use the correct error class. However, some of the errors are more iceberg specific, for example, `ALTER TABLE has no changes: missing both distribution and ordering clauses`, there isn't a corresponding Spark error class for this, and it's not practical to go to Spark to add an error class for each of the iceberg errors. Spark 4.0 requires `AnalysisException` to have error class, in another word, we don' have a way to construct an `AnalysisException` without specifying an error class , but Spark doesn't require the subclass of `AnalysisException` to have error class, that's why I am adding an `IcebergAnalysisException`, so we can still constructing an `IcebergAnalysisException` without error message only. -- 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
