actix/actix-web
1
0
Fork 0
mirror of https://github.com/fafhrd91/actix-web synced 2024-11-27 17:52:56 +01:00
Code Issues Releases Wiki Activity

do not execute blocking fn if result is not required

Browse Source
This commit is contained in:
Nikolay Kim 2019-03-07 19:31:17 -08:00
parent eef687ec80
commit 2f6df11183
1 changed files with 6 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/blocking.rs
View File

@ -41,6 +41,7 @@ thread_local! {
}; };
} }
/// Blocking operation execution error
#[derive(Debug, Display)] #[derive(Debug, Display)]
pub enum BlockingError<E: fmt::Debug> { pub enum BlockingError<E: fmt::Debug> {
#[display(fmt = "{:?}", _0)] #[display(fmt = "{:?}", _0)]
@ -62,13 +63,17 @@ where
let (tx, rx) = oneshot::channel(); let (tx, rx) = oneshot::channel();
POOL.with(|pool| { POOL.with(|pool| {
pool.execute(move || { pool.execute(move || {
let _ = tx.send(f()); if !tx.is_canceled() {
let _ = tx.send(f());
}
}) })
}); });
CpuFuture { rx } CpuFuture { rx }
} }
/// Blocking operation completion future. It resolves with results
/// of blocking function execution.
pub struct CpuFuture<I, E> { pub struct CpuFuture<I, E> {
rx: oneshot::Receiver<Result<I, E>>, rx: oneshot::Receiver<Result<I, E>>,
} }