Bring back Fail

jobs-actix/src/lib.rs

@@ -2,7 +2,7 @@ use std::{collections::BTreeMap, sync::Arc, time::Duration};
 
 use actix::{Actor, Addr, Arbiter, SyncArbiter};
 use background_jobs_core::{Job, Processor, ProcessorMap, Stats, Storage};
-use failure::Error;
+use failure::{Error, Fail};
 use futures::Future;
 
 mod every;
@@ -28,7 +28,7 @@ pub struct ServerConfig<S> {
 impl<S> ServerConfig<S>
 where
     S: Storage + Sync + 'static,
-    S::Error: Send + Sync,
+    S::Error: Fail,
 {
     /// Create a new ServerConfig
     pub fn new(storage: S) -> Self {

jobs-actix/src/storage.rs

@@ -1,5 +1,5 @@
 use background_jobs_core::{JobInfo, NewJobInfo, ReturnJobInfo, Stats, Storage};
-use failure::Error;
+use failure::{Error, Fail};
 
 pub(crate) trait ActixStorage {
     fn new_job(&mut self, job: NewJobInfo) -> Result<u64, Error>;
@@ -11,17 +11,15 @@ pub(crate) trait ActixStorage {
     fn get_stats(&self) -> Result<Stats, Error>;
 }
 
-pub(crate) struct StorageWrapper<S, E>(pub(crate) S)
+pub(crate) struct StorageWrapper<S>(pub(crate) S)
 where
-    S: Storage<Error = E>,
-    S::Error: Send,
-    E: std::error::Error + Send;
+    S: Storage,
+    S::Error: Fail;
 
-impl<S, E> ActixStorage for StorageWrapper<S, E>
+impl<S> ActixStorage for StorageWrapper<S>
 where
-    S: Storage<Error = E>,
-    S::Error: Send,
-    E: std::error::Error + Send + Sync + 'static,
+    S: Storage,
+    S::Error: Fail,
 {
     fn new_job(&mut self, job: NewJobInfo) -> Result<u64, Error> {
         self.0.new_job(job).map_err(Error::from)

jobs-core/src/storage.rs

@@ -18,6 +18,7 @@
  */
 
 use chrono::offset::Utc;
+use failure::Fail;
 use log::error;
 
 use crate::{JobInfo, NewJobInfo, ReturnJobInfo, Stats};
@@ -30,7 +31,7 @@ use crate::{JobInfo, NewJobInfo, ReturnJobInfo, Stats};
 /// the `background-jobs-sled-storage` crate.
 pub trait Storage: Clone + Send {
     /// The error type used by the storage mechansim.
-    type Error: std::error::Error;
+    type Error: Fail;
 
     /// This method generates unique IDs for jobs
     fn generate_id(&mut self) -> Result<u64, Self::Error>;
@@ -264,7 +265,7 @@ pub mod memory_storage {
         }
     }
 
-    #[derive(Clone, Debug)]
+    #[derive(Clone, Debug, Fail)]
     pub enum Never {}
 
     impl fmt::Display for Never {
@@ -273,12 +274,6 @@ pub mod memory_storage {
         }
     }
 
-    impl std::error::Error for Never {
-        fn description(&self) -> &str {
-            match *self {}
-        }
-    }
-
     #[derive(Clone, Debug, Fail)]
     #[fail(display = "Created too many storages, can't generate any more IDs")]
     pub struct TooManyStoragesError;