Xuanwo commented on PR #922: URL: https://github.com/apache/iceberg-rust/pull/922#issuecomment-2644500284
> * Makes `ErrorResponse`, `OK`, and `HttpClient` public to support escape-hatching for REST catalogs that don't 100% support the iceberg catalog response formats. Hi, I'm a bit concerned about this since it's really an internal implementation detail that we don't want to expose. > I'd like to use iceberg-rust for some Iceberg-like clients that return partial TableMetadata responses Does it make more sense to allow table metadata to be partially parsed? I'm happy to discuss this direction if you are willing to establish an issue for it. -- 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
