Skip to content

Commit d48abff

Browse files
authored
Fix failing logging test (#790)
* add logging fix * change to trace * Add sql check
1 parent d3c767d commit d48abff

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

.github/workflows/loggingTesting.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -101,7 +101,7 @@ jobs:
101101
if [ -f "$LOG_FILE" ]; then
102102
echo "Log file found. Checking contents..."
103103
104-
REQUIRED_STRINGS=("Executing HTTP request"
104+
REQUIRED_STRINGS=("sql = {SELECT 1}",
105105
"Result retrieved successfully"
106106
"Closing global async HTTP client"
107107
"Global async HTTP client has been shut down")

src/main/java/com/databricks/jdbc/dbclient/impl/sqlexec/DatabricksSdkClient.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -194,7 +194,7 @@ public DatabricksResultSet executeStatement(
194194
}
195195
LOGGER.debug(
196196
String.format(
197-
"Executing sql %s, statementType %s, compute %s, StatementID %s",
197+
"Executing sql = {%s}, statementType %s, compute %s, StatementID %s",
198198
sql, statementType, computeResource, statementId));
199199
StatementId typedStatementId = new StatementId(statementId);
200200
if (parentStatement != null) {

0 commit comments

Comments
 (0)