Avoid further setter binding when possible

As of gh-1163, the `GraphQlArgumentBinder` can bind using both a
constructor and setters. This is especially useful for Kotlin data
classes. This change introduced a significant performance penalty for
large and deep data sets.

This commit ensures that we are only performing setter binding for input
data that was not already consumed during the constructor binding phase.

Fixes gh-1284

Author: bclozel

spring-graphql/src/main/java/org/springframework/graphql/data/GraphQlArgumentBinder.java

@@ -20,6 +20,7 @@
 import java.lang.reflect.Field;
 import java.util.Collection;
 import java.util.Collections;
+import java.util.HashMap;
 import java.util.Map;
 import java.util.Optional;
 
@@ -276,6 +277,7 @@ private Map<?, Object> bindMapToMap(
 	private Object bindViaConstructorAndSetters(Constructor<?> constructor,
 			Map<String, Object> rawMap, ResolvableType ownerType, ArgumentsBindingResult bindingResult) {
 
+		Map<String, Object> dataToBind = new HashMap<>(rawMap);
 		String[] paramNames = BeanUtils.getParameterNames(constructor);
 		Class<?>[] paramTypes = constructor.getParameterTypes();
 		Object[] constructorArguments = new Object[paramTypes.length];
@@ -287,7 +289,8 @@ private Object bindViaConstructorAndSetters(Constructor<?> constructor,
 					ResolvableType.forConstructorParameter(constructor, i).getType(), ownerType);
 
 			constructorArguments[i] = bindRawValue(
-					name, rawMap.get(name), !rawMap.containsKey(name), targetType, paramTypes[i], bindingResult);
+					name, dataToBind.get(name), !dataToBind.containsKey(name), targetType, paramTypes[i], bindingResult);
+			dataToBind.remove(name);
 		}
 
 		Object target;
@@ -302,9 +305,9 @@ private Object bindViaConstructorAndSetters(Constructor<?> constructor,
 			throw ex;
 		}
 
-		// If no errors, apply setters too
-		if (!bindingResult.hasErrors()) {
-			bindViaSetters(target, rawMap, ownerType, bindingResult);
+		// If no errors and data remains to be bound, apply setters too
+		if (!dataToBind.isEmpty() && !bindingResult.hasErrors()) {
+			bindViaSetters(target, dataToBind, ownerType, bindingResult);
 		}
 
 		return target;