fix eclipse-ee4j#1408 SQL statement is printed in exceptions despite eclipselink.logging.level.sql is OFF

Signed-off-by: Ryosuke Okada <ryosuke.okada@fujitsu.com>

Author: 11rx4f

foundation/org.eclipse.persistence.core/src/main/java/org/eclipse/persistence/exceptions/DatabaseException.java

@@ -22,6 +22,7 @@
 import org.eclipse.persistence.internal.sessions.AbstractSession;
 import org.eclipse.persistence.exceptions.i18n.ExceptionMessageGenerator;
 import org.eclipse.persistence.sessions.DataRecord;
+import org.eclipse.persistence.logging.SessionLog;
 
 /**
  * <P><B>Purpose</B>:
@@ -210,7 +211,7 @@ public String getMessage() {
             } else {
                 writer.write("000");
             }
-            if (getCall() != null) {
+            if (getCall() != null && session.shouldLog(SessionLog.FINE, SessionLog.SQL)) {
                 writer.write(cr());
                 writer.write(getIndentationString());
                 writer.write(ExceptionMessageGenerator.getHeader("CallHeader"));

foundation/org.eclipse.persistence.core/src/main/java/org/eclipse/persistence/internal/sessions/AbstractSession.java

@@ -1856,6 +1856,9 @@ public Object executeQuery(DatabaseQuery query, AbstractRecord row, int retryCou
                 if (queryException.getSession() == null) {
                     queryException.setSession(this);
                 }
+                if (queryException.getQuery().getSession() == null) {
+                    queryException.getQuery().setSession(this);
+                }
             } else if (exception instanceof DatabaseException) {
                 DatabaseException databaseException = (DatabaseException)exception;
                 if (databaseException.getQuery() == null) {
@@ -1867,6 +1870,9 @@ public Object executeQuery(DatabaseQuery query, AbstractRecord row, int retryCou
                 if (databaseException.getSession() == null) {
                     databaseException.setSession(this);
                 }
+                if (databaseException.getQuery().getSession() == null) {
+                    databaseException.getQuery().setSession(this);
+                }
                 //if this query is a read query outside of a transaction then we may be able to retry the query
                 if (!isInTransaction() && query.isReadQuery() && getDatasourceLogin() instanceof DatabaseLogin) {
                     final int count = getLogin().getQueryRetryAttemptCount();

foundation/org.eclipse.persistence.core/src/main/java/org/eclipse/persistence/queries/DatabaseQuery.java

@@ -60,6 +60,7 @@
 import org.eclipse.persistence.sessions.remote.*;
 import org.eclipse.persistence.sessions.DatabaseRecord;
 import org.eclipse.persistence.sessions.SessionProfiler;
+import org.eclipse.persistence.logging.SessionLog;
 
 /**
  * <p>
@@ -2681,10 +2682,13 @@ public String toString() {
         if ((getName() != null) && (!getName().equals(""))) {
             nameString = "name=\"" + getName() + "\" ";
         }
-        if (isSQLCallQuery()) {
-            queryString = "sql=\"" + getSQLString() + "\"";
-        } else if (isJPQLCallQuery()) {
-            queryString = "jpql=\"" + getJPQLString() + "\"";
+
+        if (session.shouldLog(SessionLog.FINE, SessionLog.SQL)) {
+            if (isSQLCallQuery()) {
+                queryString = "sql=\"" + getSQLString() + "\"";
+            } else if (isJPQLCallQuery()) {
+                queryString = "jpql=\"" + getJPQLString() + "\"";
+            }
         }
         return getClass().getSimpleName() + "(" + nameString + referenceClassString + queryString + ")";
     }