jorgecarleitao commented on pull request #8453:
URL: https://github.com/apache/arrow/pull/8453#issuecomment-707790241


   Ah, I see the use-case now.
   
   Ok, I think that [tokio's recommendation 
is](https://docs.rs/tokio/0.2.22/tokio/runtime/struct.Runtime.html):
   
   > Instances of Runtime can be created using new or Builder. However, most 
users will use the #[tokio::main] annotation on their entry point instead.
   
   Thus, IMO we should remove the `#[tokio:main]` and initialize the runtime 
using its associated 
[builder](https://docs.rs/tokio/0.2.22/tokio/runtime/struct.Builder.html), to 
configure the size of the thread pool, whenever we want to expose the 
concurrency configuration.
   
   Note that in the benchmarks, we can even specify different concurrency 
parameter depending on the benchmark (in case we want to perform some scaling).
   


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to