Protei/PromucFlow_constructor
2
0
Fork 0
Code Issues Pull Requests Actions 2 Packages Projects Releases Wiki Activity

chore: Use concurrent set instead of set (#28298)

Browse Source 
## Description
Replaced set with concurrent set

#### PR fixes following issue(s)
Fixes #28322
This commit is contained in:
Nayan 2023-10-26 11:08:00 +06:00 committed by GitHub
parent 7b125741c7
commit d199b49eee
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
app/server/appsmith-server/src/main/java/com/appsmith/server/onpageload/internal/PageLoadExecutablesUtilCEImpl.java
View File

@ -34,6 +34,7 @@ import java.util.List;
import java.util.Map; import java.util.Map;
import java.util.Optional; import java.util.Optional;
import java.util.Set; import java.util.Set;
import java.util.concurrent.ConcurrentHashMap;
import java.util.concurrent.atomic.AtomicReference; import java.util.concurrent.atomic.AtomicReference;
import java.util.regex.Matcher; import java.util.regex.Matcher;
import java.util.regex.Pattern; import java.util.regex.Pattern;
@ -101,7 +102,7 @@ public class PageLoadExecutablesUtilCEImpl implements PageLoadExecutablesUtilCE
Set<String> onPageLoadExecutableSetRef = new HashSet<>(); Set<String> onPageLoadExecutableSetRef = new HashSet<>();
Set<String> explicitUserSetOnLoadExecutablesRef = new HashSet<>(); Set<String> explicitUserSetOnLoadExecutablesRef = new HashSet<>();
Set<String> bindingsFromExecutablesRef = new HashSet<>(); Set<String> bindingsFromExecutablesRef = ConcurrentHashMap.newKeySet();
// Function `extractAndSetExecutableBindingsInGraphEdges` updates this map to keep a track of all the // Function `extractAndSetExecutableBindingsInGraphEdges` updates this map to keep a track of all the
// executables which // executables which
@ -857,7 +858,7 @@ public class PageLoadExecutablesUtilCEImpl implements PageLoadExecutablesUtilCE
// All executables found from possibleExecutableNames set would add their dependencies in the following set for // All executables found from possibleExecutableNames set would add their dependencies in the following set for
// further // further
// walk to find more executables recursively. // walk to find more executables recursively.
Set<String> newBindings = new HashSet<>(); Set<String> newBindings = ConcurrentHashMap.newKeySet();
// First fetch all the executables in the page whose name matches the words found in all the dynamic bindings // First fetch all the executables in the page whose name matches the words found in all the dynamic bindings
Mono<List<EntityDependencyNode>> findAndAddExecutablesInBindingsMono = getPossibleEntityReferences( Mono<List<EntityDependencyNode>> findAndAddExecutablesInBindingsMono = getPossibleEntityReferences(