Re: [I] [DISCUSSION] `Runtime` support enhancement [iceberg-rust]

["CTTY (via GitHub)" <gi...@apache.org>]

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: issues-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org