From 3951612af94bc6dc20f78a0514afeedc211d6768 Mon Sep 17 00:00:00 2001 From: Krzysztof Nozderko Date: Tue, 11 Jun 2024 12:18:27 +0000 Subject: [PATCH] Fix review notes --- .github/workflows/main.yml | 4 ++-- Snowflake.Data/Core/Session/ConnectionPoolManager.cs | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/.github/workflows/main.yml b/.github/workflows/main.yml index 470d3ffa5..171778d11 100644 --- a/.github/workflows/main.yml +++ b/.github/workflows/main.yml @@ -3,9 +3,9 @@ name: DotNet Build and Test # Triggers the workflow on push or pull request events but only for the master branch on: push: - branches: [ master, pool/SNOW-860872-connection-pool ] + branches: [ master ] pull_request: - branches: [ master, pool/SNOW-860872-connection-pool ] + branches: [ master ] workflow_dispatch: inputs: logLevel: diff --git a/Snowflake.Data/Core/Session/ConnectionPoolManager.cs b/Snowflake.Data/Core/Session/ConnectionPoolManager.cs index fa4dde5c1..09bfa5821 100644 --- a/Snowflake.Data/Core/Session/ConnectionPoolManager.cs +++ b/Snowflake.Data/Core/Session/ConnectionPoolManager.cs @@ -122,7 +122,7 @@ public bool GetPooling() public SessionPool GetPool(string connectionString, SecureString password) { - s_logger.Debug($"ConnectionPoolManager::GetPool"); + s_logger.Debug("ConnectionPoolManager::GetPool with connection string and secure password"); var poolKey = GetPoolKey(connectionString, password); if (_pools.TryGetValue(poolKey, out var item)) @@ -147,7 +147,7 @@ public SessionPool GetPool(string connectionString, SecureString password) public SessionPool GetPool(string connectionString) { - s_logger.Debug($"ConnectionPoolManager::GetPool"); + s_logger.Debug("ConnectionPoolManager::GetPool with connection string"); return GetPool(connectionString, null); }