Skip to content

Commit

Permalink
Document implicit yield in install() per rayon-rs#1105
Browse files Browse the repository at this point in the history
  • Loading branch information
benkay86 committed Nov 29, 2023
1 parent d1b18e6 commit 8e24971
Showing 1 changed file with 35 additions and 0 deletions.
35 changes: 35 additions & 0 deletions rayon-core/src/thread_pool/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -80,6 +80,41 @@ impl ThreadPool {
/// thread-local data from the current thread will not be
/// accessible.
///
/// # Warning: execution order
///
/// `install()` implicitly calls [`ThreadPool::yield_now()`],
/// potentially scheduling another task to run on the current thread. For
/// example:
///
/// ```rust
/// # use rayon_core as rayon;
/// use rayon::iter::{IntoParallelIterator, ParallelIterator};
/// fn main() {
/// rayon::ThreadPoolBuilder::new().num_threads(1).build_global().unwrap();
/// let pool = rayon::ThreadPoolBuilder::default().build().unwrap();
/// (0..3).into_par_iter().for_each(|_| {
/// print!("one ");
/// pool.install(||{});
/// print!("two ");
/// });
/// }
/// ```
///
/// Since we configured just one thread in the global pool, one might
/// expect this to run sequentially, producing:
///
/// ```ascii
/// one two one two one two
/// ```
///
/// However each call to `install()` yields implicitly, allowing rayon to
/// run several instances of the `for_each()` loop concurrently on the
/// single, global thread. The following output would be equally valid:
///
/// ```ascii
/// one one two one two two
/// ```
///
/// # Panics
///
/// If `op` should panic, that panic will be propagated.
Expand Down

0 comments on commit 8e24971

Please sign in to comment.