Merge pull request #70 Pass logger to TxControl.ToProto from LipatovA…

…lexander/main fix: Pass logger to TxControl.ToProto
ydb-platform · Apr 23, 2024 · 4262859 · 4262859
2 parents a5c7245 + 8d1257c
commit 4262859
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/Ydb.Sdk/src/Services/Table/ExecuteDataQuery.cs b/src/Ydb.Sdk/src/Services/Table/ExecuteDataQuery.cs
@@ -65,7 +65,7 @@ public async Task<ExecuteDataQueryResponse> ExecuteDataQuery(
         {
             OperationParams = MakeOperationParams(settings),
             SessionId = Id,
-            TxControl = txControl.ToProto(),
+            TxControl = txControl.ToProto(Logger),
             Query = new Ydb.Table.Query
             {
                 YqlText = query

diff --git a/src/Ydb.Sdk/src/Services/Table/Transaction.cs b/src/Ydb.Sdk/src/Services/Table/Transaction.cs
@@ -114,7 +114,7 @@ public TxControl Commit()
         return this;
     }
 
-    internal TransactionControl ToProto(ILogger? logger = null)
+    internal TransactionControl ToProto(ILogger logger)
     {
         if (_txNum != null)
         {