Re: [PR] Move SpawnedTask from datafusion_physical_plan to new `datafusion_common_runtime` crate [arrow-datafusion]

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

mustafasrepo commented on PR #9414:
URL: https://github.com/apache/arrow-datafusion/pull/9414#issuecomment-1973228776

   > Thank you @mustafasrepo
   > 
   > Perhaps we can also re-export this new module in the datafusion core crate as well (can be a follow on PR). That way people can still use this code via the core datafusion crate (and don't have to add an explicit new dependnecy on `datafusion-common-runtime`?
   > 
   > For example,
   > 
   > https://github.com/apache/arrow-datafusion/blob/b2ff249bfb918ac6697dbc92b51262a7bdbb5971/datafusion/core/src/lib.rs#L481-L483
   > 
   > Maybe add something like
   > 
   > ```rust
   > pub mod common {
   >     pub use datafusion_common::*;
   >     pub mod runtime {
   >       pub use datafusion_common_runtime::*;
   >     }
   > }
   > ```
   
   Re-exported from core. Thanks for the suggestion. 


-- 
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: github-unsubscribe@arrow.apache.org

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