Using non-blocking optimization to compile function.

Author: lucky8987

Diff for: spring-jdbc/src/main/java/org/springframework/jdbc/core/simple/AbstractJdbcCall.java

@@ -21,6 +21,7 @@
 import java.util.List;
 import java.util.Map;
 import java.util.Set;
+import java.util.concurrent.atomic.AtomicBoolean;
 
 import javax.sql.DataSource;
 
@@ -70,7 +71,7 @@ public abstract class AbstractJdbcCall {
 	 * Has this operation been compiled? Compilation means at least checking
 	 * that a DataSource or JdbcTemplate has been provided.
 	 */
-	private volatile boolean compiled;
+	private final AtomicBoolean compiled = new AtomicBoolean(false);
 
 	/** The generated string used for call statement. */
 	private @Nullable String callString;
@@ -278,19 +279,18 @@ public void addDeclaredRowMapper(String parameterName, RowMapper<?> rowMapper) {
 	 * @throws org.springframework.dao.InvalidDataAccessApiUsageException if the object hasn't
 	 * been correctly initialized, for example if no DataSource has been provided
 	 */
-	public final synchronized void compile() throws InvalidDataAccessApiUsageException {
-		if (!isCompiled()) {
-			if (getProcedureName() == null) {
-				throw new InvalidDataAccessApiUsageException("Procedure or Function name is required");
-			}
+	public final void compile() throws InvalidDataAccessApiUsageException {
+		if (getProcedureName() == null) {
+			throw new InvalidDataAccessApiUsageException("Procedure or Function name is required");
+		}
+		if (this.compiled.compareAndSet(false, true)) {
 			try {
 				this.jdbcTemplate.afterPropertiesSet();
 			}
 			catch (IllegalArgumentException ex) {
 				throw new InvalidDataAccessApiUsageException(ex.getMessage());
 			}
 			compileInternal();
-			this.compiled = true;
 			if (logger.isDebugEnabled()) {
 				logger.debug("SqlCall for " + (isFunction() ? "function" : "procedure") +
 						" [" + getProcedureName() + "] compiled");
@@ -335,7 +335,7 @@ protected void onCompileInternal() {
 	 * @return whether this operation is compiled and ready to use
 	 */
 	public boolean isCompiled() {
-		return this.compiled;
+		return this.compiled.get();
 	}
 
 	/**

Diff for: spring-jdbc/src/main/java/org/springframework/jdbc/core/simple/AbstractJdbcInsert.java

@@ -28,6 +28,7 @@
 import java.util.List;
 import java.util.Locale;
 import java.util.Map;
+import java.util.concurrent.atomic.AtomicBoolean;
 
 import javax.sql.DataSource;
 
@@ -82,7 +83,7 @@ public abstract class AbstractJdbcInsert {
 	 * Has this operation been compiled? Compilation means at least checking
 	 * that a DataSource or JdbcTemplate has been provided.
 	 */
-	private volatile boolean compiled;
+	private final AtomicBoolean compiled = new AtomicBoolean(false);
 
 	/** The generated string used for insert statement. */
 	private String insertString = "";
@@ -268,23 +269,22 @@ public boolean isQuoteIdentifiers() {
 	 * @throws InvalidDataAccessApiUsageException if the object hasn't been correctly initialized,
 	 * for example if no DataSource has been provided
 	 */
-	public final synchronized void compile() throws InvalidDataAccessApiUsageException {
-		if (!isCompiled()) {
-			if (getTableName() == null) {
-				throw new InvalidDataAccessApiUsageException("Table name is required");
-			}
-			if (isQuoteIdentifiers() && this.declaredColumns.isEmpty()) {
-				throw new InvalidDataAccessApiUsageException(
-						"Explicit column names must be provided when using quoted identifiers");
-			}
+	public final void compile() throws InvalidDataAccessApiUsageException {
+		if (getTableName() == null) {
+			throw new InvalidDataAccessApiUsageException("Table name is required");
+		}
+		if (isQuoteIdentifiers() && this.declaredColumns.isEmpty()) {
+			throw new InvalidDataAccessApiUsageException(
+					"Explicit column names must be provided when using quoted identifiers");
+		}
+		if (this.compiled.compareAndSet(false, true)) {
 			try {
 				this.jdbcTemplate.afterPropertiesSet();
 			}
 			catch (IllegalArgumentException ex) {
 				throw new InvalidDataAccessApiUsageException(ex.getMessage());
 			}
 			compileInternal();
-			this.compiled = true;
 			if (logger.isDebugEnabled()) {
 				logger.debug("JdbcInsert for table [" + getTableName() + "] compiled");
 			}
@@ -320,7 +320,7 @@ protected void onCompileInternal() {
 	 * @return whether this operation is compiled and ready to use
 	 */
 	public boolean isCompiled() {
-		return this.compiled;
+		return this.compiled.get();
 	}
 
 	/**