This is an automated email from the ASF dual-hosted git repository.

zhangliang pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/shardingsphere.git


The following commit(s) were added to refs/heads/master by this push:
     new 1fd45b3  refine synchronized block (#7958)
1fd45b3 is described below

commit 1fd45b306e5b9b073e7c895fac69ca616fc5e256
Author: Zhang Yonglun <[email protected]>
AuthorDate: Thu Oct 29 16:52:15 2020 +0800

    refine synchronized block (#7958)
    
    * refine synchronized block
    
    * for comments
---
 .../apache/shardingsphere/sql/parser/api/SQLParserEngine.java    | 9 +--------
 1 file changed, 1 insertion(+), 8 deletions(-)

diff --git 
a/shardingsphere-sql-parser/shardingsphere-sql-parser-engine/src/main/java/org/apache/shardingsphere/sql/parser/api/SQLParserEngine.java
 
b/shardingsphere-sql-parser/shardingsphere-sql-parser-engine/src/main/java/org/apache/shardingsphere/sql/parser/api/SQLParserEngine.java
index 12a1647..d73a0e3 100644
--- 
a/shardingsphere-sql-parser/shardingsphere-sql-parser-engine/src/main/java/org/apache/shardingsphere/sql/parser/api/SQLParserEngine.java
+++ 
b/shardingsphere-sql-parser/shardingsphere-sql-parser-engine/src/main/java/org/apache/shardingsphere/sql/parser/api/SQLParserEngine.java
@@ -68,13 +68,6 @@ public final class SQLParserEngine {
         if (ENGINES.containsKey(databaseType)) {
             return ENGINES.get(databaseType);
         }
-        synchronized (ENGINES) {
-            if (ENGINES.containsKey(databaseType)) {
-                return ENGINES.get(databaseType);
-            }
-            SQLParserExecutor result = new SQLParserExecutor(databaseType);
-            ENGINES.put(databaseType, result);
-            return result;
-        }
+        return ENGINES.computeIfAbsent(databaseType, key -> new 
SQLParserExecutor(databaseType));
     }
 }

Reply via email to