huaxingao opened a new pull request, #10766: URL: https://github.com/apache/iceberg/pull/10766
In Iceberg's `spark-extensions`, if we encounter any invalid SQL statements, we throw Spark's `AnalysisException`. However, it would be better to have Iceberg's own `IcebergAnalysisException` to differentiate it from the `AnalysisException` issued by Spark itself. Another reason to have Iceberg's own `IcebergAnalysisException` is due to Spark's new error handling scheme. Since Spark 3.2, Spark has introduced an error class system. For the `AnalysisExceptions` that are issued in Iceberg's `spark-extensions`, we don't really have a corresponding Spark error class. Therefore, it seems better to have Iceberg's own `IcebergAnalysisException` to differentiate it from the `AnalysisException` issued by Spark itself. -- 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
