Xuanwo commented on PR #16: URL: https://github.com/apache/iceberg-rust/pull/16#issuecomment-1663927396
> There is a lot of stuff in here that will be duplicated. If we want to avoid manually writing a client for the rest catalog, then this is necessary. Rust is unlike to java (which has runtime reflact) or python (which is dynamic), we must declare structs or types first before we can using them. The convertion could be simple and zero cost. > But we should be careful that don't expose these things as public api to end user. `iceberg-rest-catalog-client` is an underlying crate may be used by `iceberg` or any other users. `iceberg` can use `iceberg-rest-catalog-client` to implement `rest-catalog` support. And other users may use this client to connect to a rest catalog service directly. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
