Transfer URI Correctly (#2265)

* transfer uri regardless of queryrunner presence

* remove extra space
This commit is contained in:
Christopher Suh
2023-10-09 11:18:51 -07:00
committed by GitHub
parent dacddd250c
commit f0d3a0d546
2 changed files with 2 additions and 1 deletions

View File

@@ -925,6 +925,7 @@ namespace Microsoft.SqlTools.ServiceLayer.Connection
{ {
return false; return false;
} }
info.OwnerUri = newOwnerUri;
OwnerToConnectionMap.Remove(originalOwnerUri); OwnerToConnectionMap.Remove(originalOwnerUri);
OwnerToConnectionMap.Add(newOwnerUri, info); OwnerToConnectionMap.Add(newOwnerUri, info);
return true; return true;

View File

@@ -363,13 +363,13 @@ namespace Microsoft.SqlTools.ServiceLayer.QueryExecution
{ {
string OriginalOwnerUri = changeUriParams.OriginalOwnerUri; string OriginalOwnerUri = changeUriParams.OriginalOwnerUri;
string NewOwnerUri = changeUriParams.NewOwnerUri; string NewOwnerUri = changeUriParams.NewOwnerUri;
ConnectionService.ReplaceUri(OriginalOwnerUri, NewOwnerUri);
// Attempt to load the query // Attempt to load the query
Query query; Query query;
if (!ActiveQueries.TryRemove(OriginalOwnerUri, out query)) if (!ActiveQueries.TryRemove(OriginalOwnerUri, out query))
{ {
throw new Exception("Uri: " + OriginalOwnerUri + " is not associated with an active query."); throw new Exception("Uri: " + OriginalOwnerUri + " is not associated with an active query.");
} }
ConnectionService.ReplaceUri(OriginalOwnerUri, NewOwnerUri);
query.ConnectionOwnerURI = NewOwnerUri; query.ConnectionOwnerURI = NewOwnerUri;
ActiveQueries.TryAdd(NewOwnerUri, query); ActiveQueries.TryAdd(NewOwnerUri, query);
return Task.FromResult(true); return Task.FromResult(true);