Xuanwo commented on issue #1038: URL: https://github.com/apache/iceberg-rust/issues/1038#issuecomment-2722328079
> In that case then I think it is reasonable to make the current `ErrorKind` into a more descriptive type than a couple of discriminants (i.e. add fields to some of the variants), and we would not lose any expression. Hi, I'm still not in favor of an idea like this. `ErrorKind` itself is simple, clear, `Copy`, and easier to use in bindings like python and c. Developers (including us) can easily add context, and users can check and use it effortlessly. I didn't see good reasons to make such 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
