graphql: unlock GRAPHQL_VALIDATION_CACHE quickly

Author: kamilkisiela

Cargo.lock

@@ -7,7 +7,7 @@ edition = "2021"
 crossbeam = "0.8"
 graph = { path = "../graph" }
 graphql-parser = "0.4.0"
-graphql-tools = "0.0.19"
+graphql-tools = { git = "https://github.com/dotansimha/graphql-tools-rs", branch = "kamilkisiela-patch-1" }
 indexmap = "1.9"
 Inflector = "0.11.3"
 lazy_static = "1.2.0"

graphql/Cargo.toml

@@ -143,6 +143,23 @@ pub struct Query {
     pub query_id: String,
 }
 
+fn validate_query(query: &GraphDataQuery, document: &s::Document) -> Vec<ValidationError> {
+    let mut cache = GRAPHQL_VALIDATION_CACHE
+            .lock()
+            .unwrap_or_else(PoisonError::into_inner);
+
+    let errors = cache.entry(query.shape_hash)
+            .or_insert_with(|| {
+                validate(
+                    &document,
+                    &query.document,
+                    &GRAPHQL_VALIDATION_PLAN,
+                )
+            });
+
+    return errors.clone();
+}
+
 impl Query {
     /// Process the raw GraphQL query `query` and prepare for executing it.
     /// The returned `Query` has already been validated and, if `max_complexity`
@@ -156,17 +173,7 @@ impl Query {
         max_complexity: Option<u64>,
         max_depth: u8,
     ) -> Result<Arc<Self>, Vec<QueryExecutionError>> {
-        GRAPHQL_VALIDATION_CACHE
-            .lock()
-            .unwrap_or_else(PoisonError::into_inner)
-            .entry(query.shape_hash)
-            .or_insert_with(|| {
-                validate(
-                    &schema.document(),
-                    &query.document,
-                    &GRAPHQL_VALIDATION_PLAN,
-                )
-            });
+        let validation_errors = validate_query(&query, &schema.document());
 
         if !validation_errors.is_empty() {
             if !ENV_VARS.graphql.silent_graphql_validations {