More thread safety improvements

Author: PicoCreator

src/main/java/picoded/dstack/file/simple/FileSimple_FileWorkspaceMap.java

@@ -23,7 +23,7 @@ public class FileSimple_FileWorkspaceMap extends Core_FileWorkspaceMap {
 	//--------------------------------------------------------------------------
 
 	/// The file directory to opreate from
-	protected File baseDir = null;
+	protected final File baseDir;
 
 	/// The file suffix to use for JSON object records
 

src/main/java/picoded/dstack/hazelcast/core/HazelcastStack.java

@@ -29,7 +29,7 @@ public abstract class HazelcastStack extends CoreStack {
 	/**
 	 * The internal JSql connection
 	 */
-	protected HazelcastInstance conn = null;
+	protected final HazelcastInstance conn;
 
 	/**
 	 * Constructor with configuration map

src/main/java/picoded/dstack/jsql/JSqlStack.java

@@ -15,7 +15,7 @@ public class JSqlStack extends CoreStack {
 	/**
 	 * The internal JSql connection
 	 */
-	protected final JSql conn ;
+	protected final JSql conn;
 
 	/**
 	 * Constructor with configuration map

src/main/java/picoded/dstack/mongodb/MongoDBStack.java

@@ -33,15 +33,15 @@ public class MongoDBStack extends CoreStack {
 	/**
 	 * The internal MongoClient connection
 	 */
-	protected final MongoClient client_conn ;
-	protected final MongoDatabase db_conn ;
+	protected final MongoClient client_conn;
+	protected final MongoDatabase db_conn;
 
 	/**
 	 * The secondary connetion settings
 	 */
-	protected final MongoClient sec_client_conn ;
-	protected final MongoDatabase sec_db_conn ;
-	protected final String sec_mode ;
+	protected final MongoClient sec_client_conn;
+	protected final MongoDatabase sec_db_conn;
+	protected final String sec_mode;
 
 	//-------------------------------------------------------------------------
 	// Connector utilities

src/main/java/picoded/dstack/redisson/RedissonStack.java

@@ -22,7 +22,7 @@ public class RedissonStack extends CoreStack {
 	 * The internal Redis connection
 	 */
 
-	protected RedissonClient conn = null;
+	protected final RedissonClient conn;
 
 	//-------------------------------------------------------------------------
 	// Database connection constructor

src/main/java/picoded/dstack/struct/simple/StructSimple_DataObjectMap.java

@@ -30,7 +30,7 @@ public class StructSimple_DataObjectMap extends Core_DataObjectMap_struct {
 	/**
 	 * Stores the key to value map
 	 **/
-	protected Map<String, Map<String, Object>> valueMap = new ConcurrentHashMap<String, Map<String, Object>>();
+	protected final Map<String, Map<String, Object>> valueMap = new ConcurrentHashMap<String, Map<String, Object>>();
 
 	/**
 	 * @return Storage map used centrally for all operations

src/main/java/picoded/dstack/struct/simple/StructSimple_FileWorkspaceMap.java

@@ -17,10 +17,10 @@
 public class StructSimple_FileWorkspaceMap extends Core_FileWorkspaceMap {
 
 	// Stores all the various data for structSimple
-	protected ConcurrentHashMap<String, ConcurrentHashMap<String, byte[]>> fileContentMap = new ConcurrentHashMap<String, ConcurrentHashMap<String, byte[]>>();
+	protected final ConcurrentHashMap<String, ConcurrentHashMap<String, byte[]>> fileContentMap = new ConcurrentHashMap<String, ConcurrentHashMap<String, byte[]>>();
 
 	// Handles read / write locks
-	protected ReentrantReadWriteLock accessLock = new ReentrantReadWriteLock();
+	protected final ReentrantReadWriteLock accessLock = new ReentrantReadWriteLock();
 
 	//--------------------------------------------------------------------------
 	//

src/main/java/picoded/dstack/struct/simple/StructSimple_KeyLongMap.java

@@ -20,17 +20,17 @@ public class StructSimple_KeyLongMap extends Core_KeyLongMap {
 	/**
 	 * Stores the key to long map
 	 **/
-	protected ConcurrentMap<String, Long> longMap = new ConcurrentHashMap<String, Long>();
+	protected final ConcurrentMap<String, Long> longMap = new ConcurrentHashMap<String, Long>();
 
 	/**
 	 * Stores the expire timestamp
 	 **/
-	protected ConcurrentMap<String, Long> expireMap = new ConcurrentHashMap<String, Long>();
+	protected final ConcurrentMap<String, Long> expireMap = new ConcurrentHashMap<String, Long>();
 
 	/**
 	 * Read write lock
 	 **/
-	protected ReentrantReadWriteLock accessLock = new ReentrantReadWriteLock();
+	protected final ReentrantReadWriteLock accessLock = new ReentrantReadWriteLock();
 
 	//--------------------------------------------------------------------------
 	//

src/main/java/picoded/dstack/struct/simple/StructSimple_KeyValueMap.java

@@ -29,17 +29,17 @@ public class StructSimple_KeyValueMap extends Core_KeyValueMap {
 	/**
 	 * Stores the key to value map
 	 **/
-	protected ConcurrentMap<String, String> valueMap = new ConcurrentHashMap<String, String>();
+	protected final ConcurrentMap<String, String> valueMap = new ConcurrentHashMap<String, String>();
 
 	/**
 	 * Stores the expire timestamp
 	 **/
-	protected ConcurrentMap<String, Long> expireMap = new ConcurrentHashMap<String, Long>();
+	protected final ConcurrentMap<String, Long> expireMap = new ConcurrentHashMap<String, Long>();
 
 	/**
 	 * Read write lock
 	 **/
-	protected ReentrantReadWriteLock accessLock = new ReentrantReadWriteLock();
+	protected final ReentrantReadWriteLock accessLock = new ReentrantReadWriteLock();
 
 	//--------------------------------------------------------------------------
 	//