change way we show query plan (microsoft#2866)

msnidhijangir · Oct 16, 2018 · ac1f754 · ac1f754
1 parent 225d168
commit ac1f754
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/src/sql/parts/query/editor/queryResultsView.ts b/src/sql/parts/query/editor/queryResultsView.ts
@@ -203,12 +203,12 @@ export class QueryResultsView {
 			if (!this._panelView.contains(this.qpTab)) {
 				this._panelView.pushTab(this.qpTab);
 			}
-		} else if (queryRunner.isQueryPlan) {
-			let disp = queryRunner.onResultSet(() => {
-				this.showPlan(queryRunner.planXml);
-				disp.dispose();
-			});
 		}
+		this.runnerDisposables.push(queryRunner.onResultSet(() => {
+			if (queryRunner.isQueryPlan) {
+				this.showPlan(queryRunner.planXml);
+			}
+		}));
 		if (this.input.state.activeTab) {
 			this._panelView.showTab(this.input.state.activeTab);
 		}