YT-23616: Move creation of client context to separate function

hiddenpath · hiddenpath · commit 63e9f6ad8914 · 2025-02-15T17:16:22.000+03:00
commit_hash:00566ef65ebff8cb4a46a4ffaa1772007fd47fab
diff --git a/yt/cpp/mapreduce/client/client.cpp b/yt/cpp/mapreduce/client/client.cpp
@@ -1496,7 +1496,7 @@ void TClient::CheckShutdown() const
     }
 }
 
-TClientPtr CreateClientImpl(
+TClientContext CreateClientContext(
     const TString& serverName,
     const TCreateClientOptions& options)
 {
@@ -1566,6 +1566,15 @@ TClientPtr CreateClientImpl(
         TConfig::ValidateToken(context.Token);
     }
 
+    return context;
+}
+
+TClientPtr CreateClientImpl(
+    const TString& serverName,
+    const TCreateClientOptions& options)
+{
+    auto context = CreateClientContext(serverName, options);
+
     auto globalTxId = GetGuid(context.Config->GlobalTxId);
 
     auto retryConfigProvider = options.RetryConfigProvider_;

Original file line number	Diff line number	Diff line change
`@@ -1496,7 +1496,7 @@ void TClient::CheckShutdown() const`
`1496`	`1496`	`}`
`1497`	`1497`	`}`
`1498`	`1498`
`1499`		`-TClientPtr CreateClientImpl(`
	`1499`	`+TClientContext CreateClientContext(`
`1500`	`1500`	`const TString& serverName,`
`1501`	`1501`	`const TCreateClientOptions& options)`
`1502`	`1502`	`{`
`@@ -1566,6 +1566,15 @@ TClientPtr CreateClientImpl(`
`1566`	`1566`	`TConfig::ValidateToken(context.Token);`
`1567`	`1567`	`}`
`1568`	`1568`
	`1569`	`+ return context;`
	`1570`	`+}`
	`1571`	`+`
	`1572`	`+TClientPtr CreateClientImpl(`
	`1573`	`+ const TString& serverName,`
	`1574`	`+ const TCreateClientOptions& options)`
	`1575`	`+{`
	`1576`	`+ auto context = CreateClientContext(serverName, options);`
	`1577`	`+`
`1569`	`1578`	`auto globalTxId = GetGuid(context.Config->GlobalTxId);`
`1570`	`1579`
`1571`	`1580`	`auto retryConfigProvider = options.RetryConfigProvider_;`