szehon-ho commented on PR #10766: URL: https://github.com/apache/iceberg/pull/10766#issuecomment-2246856254
Hi @huaxingao thanks, this mostly makes sense to me, but just wanted to get the motivation. Is it so the user can see the error comes from Iceberg vs other part of Spark? Also, is there an issue if we dont use Spark's error class system (I guess IcebergAnalysisException wont be a part of it)? also cc @RussellSpitzer if any thoughts on this -- 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
