graph: Wrap ParseError in an Arc for cloning

Author: leoyvens

graph/src/data/query/error.rs

@@ -260,23 +260,13 @@ impl From<StoreError> for QueryExecutionError {
 }
 
 /// Error caused while processing a [Query](struct.Query.html) request.
-#[derive(Debug)]
+#[derive(Clone, Debug)]
 pub enum QueryError {
     EncodingError(FromUtf8Error),
-    ParseError(failure::Error),
+    ParseError(Arc<anyhow::Error>),
     ExecutionError(QueryExecutionError),
 }
 
-impl Clone for QueryError {
-    fn clone(&self) -> Self {
-        match self {
-            QueryError::EncodingError(e) => QueryError::EncodingError(e.clone()),
-            QueryError::ParseError(e) => QueryError::ParseError(failure::err_msg(e.to_string())),
-            QueryError::ExecutionError(e) => QueryError::ExecutionError(e.clone()),
-        }
-    }
-}
-
 impl From<FromUtf8Error> for QueryError {
     fn from(e: FromUtf8Error) -> Self {
         QueryError::EncodingError(e)

server/http/src/request.rs

@@ -48,8 +48,9 @@ impl Future for GraphQLRequest {
         })?;
 
         // Parse the "query" field of the JSON body
-        let document = graphql_parser::parse_query(query_string)
-            .map_err(|e| GraphQLServerError::from(QueryError::ParseError(e.into())))?;
+        let document = graphql_parser::parse_query(query_string).map_err(|e| {
+            GraphQLServerError::from(QueryError::ParseError(Arc::new(e.compat().into())))
+        })?;
 
         // Parse the "variables" field of the JSON body, if present
         let variables = match obj.get("variables") {

server/index-node/src/request.rs

@@ -49,8 +49,9 @@ impl Future for IndexNodeRequest {
         })?;
 
         // Parse the "query" field of the JSON body
-        let document = graphql_parser::parse_query(query_string)
-            .map_err(|e| GraphQLServerError::from(QueryError::ParseError(e.into())))?;
+        let document = graphql_parser::parse_query(query_string).map_err(|e| {
+            GraphQLServerError::from(QueryError::ParseError(Arc::new(e.compat().into())))
+        })?;
 
         // Parse the "variables" field of the JSON body, if present
         let variables = match obj.get("variables") {