CTTY commented on issue #1958: URL: https://github.com/apache/iceberg-rust/issues/1958#issuecomment-4174055005
After discussing in the last rust community sync, we decide to not go with trait approach for now. but to add `IcebergRuntime` to wrap around the inner tokio runtime, and allow users to inject their own `IcebergRuntime` to the `Catalog`. We believe: - tokio runtime is still the prevalent runtime, it's hard to justify the use case of using a different runtime as of now - With the wrapper approach, the inner tokio runtime would be customizable - we provide a centralized place to access runtime with this approach, even if we have a better runtime option in the future, we can easily switch over to the trait approach -- 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]
