Fix performance bug with large number of unnamed parameters

On some occasions where col in (:args) contain a really lot args, 10k+ for instance, this commit fixes a performance (high CPU) bug by NOT traversing the whole map in basically O(n^2) manner

Signed-off-by: Mikhail Fedorov <[email protected]>

Squashed by Jens Schauder

Original pull request #2050

Author: tttomat19

spring-data-jdbc/src/main/java/org/springframework/data/jdbc/core/convert/QueryMapper.java

@@ -53,6 +53,7 @@
  * @author Mark Paluch
  * @author Jens Schauder
  * @author Yan Qiang
+ * @author Mikhail Fedorov
  * @since 3.0
  */
 public class QueryMapper {
@@ -632,7 +633,7 @@ private static String getUniqueName(MapSqlParameterSource parameterSource, Strin
 			return name;
 		}
 
-		int counter = 1;
+		int counter = values.size();
 		String uniqueName;
 
 		do {

spring-data-jdbc/src/test/java/org/springframework/data/jdbc/core/convert/QueryMapperUnitTests.java

@@ -46,6 +46,7 @@
  *
  * @author Mark Paluch
  * @author Jens Schauder
+ * @author Mikhail Fedorov
  */
 public class QueryMapperUnitTests {
 
@@ -121,7 +122,7 @@ public void shouldMapNestedGroup() {
 		Condition condition = map(criteria);
 
 		assertThat(condition).hasToString(
-				"(person.\"NAME\" = ?[:name]) AND (person.\"NAME\" = ?[:name1] OR person.age < ?[:age] OR (person.\"NAME\" != ?[:name2] AND person.age > ?[:age1]))");
+				"(person.\"NAME\" = ?[:name]) AND (person.\"NAME\" = ?[:name1] OR person.age < ?[:age] OR (person.\"NAME\" != ?[:name3] AND person.age > ?[:age4]))");
 	}
 
 	@Test // DATAJDBC-318