fixing provider config thread safety (WIP)

PicoCreator · PicoCreator · commit b0d4a71ac0b4 · 2022-08-30T10:29:47.000Z
diff --git a/src/main/java/picoded/dstack/stack/ProviderConfig.java b/src/main/java/picoded/dstack/stack/ProviderConfig.java
@@ -118,7 +118,7 @@ protected GenericConvertMap<String, Object> getStackConfig(String name) {
 	/**
 	 * Stores the respective stack providers
 	 */
-	protected final ConcurrentHashMap<String, CoreStack> providerStackMap = new ConcurrentHashMap<>();
+	protected volatile ConcurrentHashMap<String, CoreStack> providerStackMap = new ConcurrentHashMap<>();
 	
 	/**
 	 * Get the stack of the provider specified by the name,
@@ -158,7 +158,6 @@ public CoreStack getProviderStack(String name) {
 			// Return result
 			return cache;
 		}
-		
 	}
 	
 	/**

Original file line number	Diff line number	Diff line change
`@@ -118,7 +118,7 @@ protected GenericConvertMap<String, Object> getStackConfig(String name) {`
`118`	`118`	`/**`
`119`	`119`	`* Stores the respective stack providers`
`120`	`120`	`*/`
`121`		`- protected final ConcurrentHashMap<String, CoreStack> providerStackMap = new ConcurrentHashMap<>();`
	`121`	`+ protected volatile ConcurrentHashMap<String, CoreStack> providerStackMap = new ConcurrentHashMap<>();`
`122`	`122`
`123`	`123`	`/**`
`124`	`124`	`* Get the stack of the provider specified by the name,`
`@@ -158,7 +158,6 @@ public CoreStack getProviderStack(String name) {`
`158`	`158`	`// Return result`
`159`	`159`	`return cache;`
`160`	`160`	`}`
`161`		`-`
`162`	`161`	`}`
`163`	`162`
`164`	`163`	`/**`