ckaczor/azuredatastudio
1
0
Fork 0
mirror of https://github.com/ckaczor/azuredatastudio.git synced 2026-01-24 17:23:05 -05:00
Code Issues Packages Projects Releases Wiki Activity

improvement for the editor crash issue (#17278)

Browse Source 
* handle the query editor crash issue

* update message
This commit is contained in:
Alan Ren
2021-10-06 09:45:53 -07:00
committed by GitHub
parent b5b57a56c4
commit 458c4cff39
2 changed files with 20 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
src/sql/workbench/contrib/query/browser/queryResultsView.ts
View File

@@ -23,6 +23,8 @@ import { IThemeService } from 'vs/platform/theme/common/themeService';
import { Event } from 'vs/base/common/event';
import { URI } from 'vs/base/common/uri';
import { attachTabbedPanelStyler } from 'sql/workbench/common/styler';
import { INotificationService } from 'vs/platform/notification/common/notification';
import { ILogService } from 'vs/platform/log/common/log';
class MessagesView extends Disposable implements IPanelView {
private messagePanel: MessagePanel;
@@ -170,7 +172,9 @@ export class QueryResultsView extends Disposable {
container: HTMLElement,
@IThemeService themeService: IThemeService,
@IInstantiationService private instantiationService: IInstantiationService,
@IQueryModelService private queryModelService: IQueryModelService
@IQueryModelService private queryModelService: IQueryModelService,
@INotificationService private notificationService: INotificationService,
@ILogService private logService: ILogService
) {
super();
this.resultsTab = this._register(new ResultsTab(instantiationService));
@@ -312,6 +316,13 @@ export class QueryResultsView extends Disposable {
dynamicTab.captureState(input.state.dynamicModelViewTabsState);
});
let info = this.queryModelService._getQueryInfo(input.uri) || this.queryModelService._getQueryInfo(URI.parse(input.uri).toString(true));
if (info?.queryRunner?.isDisposed) {
this.logService.error(`The query runner for '${input.uri}' has been disposed.`);
this.notificationService.error(nls.localize('queryResults.queryEditorCrashError', "The query editor ran into an issue and has stopped working. Please save and reopen it."));
return;
}
if (info?.queryRunner) {
this.setQueryRunner(info.queryRunner);
} else {

8
src/sql/workbench/services/query/common/queryRunner.ts
View File

@@ -104,6 +104,12 @@ export default class QueryRunner extends Disposable {
return this._hasCompleted;
}
private _isDisposed: boolean = false;
get isDisposed(): boolean {
return this._isDisposed;
}
/**
* For public use only, for private use, directly access the member
*/
@@ -417,8 +423,10 @@ export default class QueryRunner extends Disposable {
}
public override dispose() {
this.logService.info(`Disposing the query runner of: '${this.uri}'', call stack: ${new Error().stack}`);
this._batchSets = undefined!;
super.dispose();
this._isDisposed = true;
}
public changeConnectionUri(oldUri: string, newUri: string): Promise<void> {