SONARPY-4039 Fix nondeterminism in AWS lambda handler indexing (#1057)

guillaume-dequenne · sonartech · commit 84cfd402a455 · 2026-04-10T12:46:31.000Z
GitOrigin-RevId: 49b51d11d4f47b20f992dac1ac231904dae9b111
diff --git a/python-frontend/src/main/java/org/sonar/python/project/config/ProjectConfigurationBuilder.java b/python-frontend/src/main/java/org/sonar/python/project/config/ProjectConfigurationBuilder.java
@@ -17,7 +17,6 @@
 package org.sonar.python.project.config;
 
 import java.util.Collection;
-import java.util.HashSet;
 import java.util.Map;
 import java.util.Set;
 import java.util.concurrent.ConcurrentHashMap;
@@ -38,7 +37,7 @@ public ProjectConfigurationBuilder() {
   }
 
   public ProjectConfigurationBuilder addAwsLambdaHandler(String packageName, String fullyQualifiedName) {
-    awsLambdaHandlersByPackage.computeIfAbsent(packageName, k -> new HashSet<>()).add(fullyQualifiedName);
+    awsLambdaHandlersByPackage.computeIfAbsent(packageName, k -> ConcurrentHashMap.newKeySet()).add(fullyQualifiedName);
     return this;
   }
 

Original file line number	Diff line number	Diff line change
`@@ -17,7 +17,6 @@`
`17`	`17`	`package org.sonar.python.project.config;`
`18`	`18`
`19`	`19`	`import java.util.Collection;`
`20`		`-import java.util.HashSet;`
`21`	`20`	`import java.util.Map;`
`22`	`21`	`import java.util.Set;`
`23`	`22`	`import java.util.concurrent.ConcurrentHashMap;`
`@@ -38,7 +37,7 @@ public ProjectConfigurationBuilder() {`
`38`	`37`	`}`
`39`	`38`
`40`	`39`	`public ProjectConfigurationBuilder addAwsLambdaHandler(String packageName, String fullyQualifiedName) {`
`41`		`- awsLambdaHandlersByPackage.computeIfAbsent(packageName, k -> new HashSet<>()).add(fullyQualifiedName);`
	`40`	`+ awsLambdaHandlersByPackage.computeIfAbsent(packageName, k -> ConcurrentHashMap.newKeySet()).add(fullyQualifiedName);`
`42`	`41`	`return this;`
`43`	`42`	`}`
`44`	`43`