Merge pull request #70 Pass logger to TxControl.ToProto from LipatovAlexander/main

rekby · web-flow · commit 42628591084e · 2024-04-23T15:56:52.000+03:00
fix: Pass logger to TxControl.ToProto
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)
         {

Original file line number	Diff line number	Diff line change
`@@ -65,7 +65,7 @@ public async Task<ExecuteDataQueryResponse> ExecuteDataQuery(`
`65`	`65`	`{`
`66`	`66`	`OperationParams = MakeOperationParams(settings),`
`67`	`67`	`SessionId = Id,`
`68`		`- TxControl = txControl.ToProto(),`
	`68`	`+ TxControl = txControl.ToProto(Logger),`
`69`	`69`	`Query = new Ydb.Table.Query`
`70`	`70`	`{`
`71`	`71`	`YqlText = query`
Original file line number	Diff line number	Diff line change
`@@ -114,7 +114,7 @@ public TxControl Commit()`
`114`	`114`	`return this;`
`115`	`115`	`}`
`116`	`116`
`117`		`- internal TransactionControl ToProto(ILogger? logger = null)`
	`117`	`+ internal TransactionControl ToProto(ILogger logger)`
`118`	`118`	`{`
`119`	`119`	`if (_txNum != null)`
`120`	`120`	`{`