yume/juniper
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Don't clone fragment names in executor

Browse source
This commit is contained in:
Magnus Hallin 2016-12-28 11:13:45 +01:00
parent 96c1ad0ef9
commit 6a4b883b97
1 changed files with 3 additions and 3 deletions

6
src/executor.rs
View file

@ -35,7 +35,7 @@ pub enum FieldPath<'a> {
/// The executor helps drive the query execution in a schema. It keeps track /// The executor helps drive the query execution in a schema. It keeps track
/// of the current field stack, context, variables, and errors. /// of the current field stack, context, variables, and errors.
pub struct Executor<'a, CtxT> where CtxT: 'a { pub struct Executor<'a, CtxT> where CtxT: 'a {
fragments: &'a HashMap<String, Fragment>, fragments: &'a HashMap<&'a str, &'a Fragment>,
variables: &'a HashMap<String, InputValue>, variables: &'a HashMap<String, InputValue>,
current_selection_set: Option<Vec<Selection>>, current_selection_set: Option<Vec<Selection>>,
schema: &'a SchemaType, schema: &'a SchemaType,
@ -232,7 +232,7 @@ impl<'a, CtxT> Executor<'a, CtxT> {
#[doc(hidden)] #[doc(hidden)]
pub fn fragment_by_name(&self, name: &str) -> Option<&'a Fragment> { pub fn fragment_by_name(&self, name: &str) -> Option<&'a Fragment> {
self.fragments.get(name) self.fragments.get(name).map(|f| *f)
} }
/// Add an error to the execution engine /// Add an error to the execution engine
@ -337,7 +337,7 @@ pub fn execute_validated_query<'a, QueryT, MutationT, CtxT>(
{ {
let executor = Executor { let executor = Executor {
fragments: &fragments.into_iter().map(|f| (f.item.name.item.clone(), f.item)).collect(), fragments: &fragments.iter().map(|f| (f.item.name.item.as_str(), &f.item)).collect(),
variables: variables, variables: variables,
current_selection_set: Some(op.item.selection_set), current_selection_set: Some(op.item.selection_set),
schema: &root_node.schema, schema: &root_node.schema,