From e186ffa3f20e8433019eb801c791ea2f89415b67 Mon Sep 17 00:00:00 2001 From: Sharon Ravindran Date: Fri, 6 Jan 2017 12:17:48 -0800 Subject: [PATCH] Reuse existing ServerConnection in PeekDefinition (#209) --- .../LanguageServices/PeekDefinition.cs | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/src/Microsoft.SqlTools.ServiceLayer/LanguageServices/PeekDefinition.cs b/src/Microsoft.SqlTools.ServiceLayer/LanguageServices/PeekDefinition.cs index 0f556944..d6456307 100644 --- a/src/Microsoft.SqlTools.ServiceLayer/LanguageServices/PeekDefinition.cs +++ b/src/Microsoft.SqlTools.ServiceLayer/LanguageServices/PeekDefinition.cs @@ -62,12 +62,9 @@ namespace Microsoft.SqlTools.ServiceLayer.LanguageServices { try { - // Get server object from connection - SqlConnection sqlConn = new SqlConnection(this.serverConnection.ConnectionString); - sqlConn.Open(); - ServerConnection peekConnection = new ServerConnection(sqlConn); - Server server = new Server(peekConnection); - this.database = new Database(server, peekConnection.DatabaseName); + // Reuse existing connection + Server server = new Server(this.serverConnection); + this.database = new Database(server, this.serverConnection.DatabaseName); } catch (ConnectionFailureException cfe) {