ckaczor/azuredatastudio
1
0
Fork 0
mirror of https://github.com/ckaczor/azuredatastudio.git synced 2026-02-16 18:46:40 -05:00
Code Issues Packages Projects Releases Wiki Activity

Add timeout delay to Notebook cell connect (#18475)

Browse Source
This commit is contained in:
Charles Gagnon
2022-02-18 09:05:13 -08:00
committed by GitHub
parent 8da49ff14e
commit a1a1793221
1 changed files with 9 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
src/sql/workbench/contrib/notebook/browser/cellViews/code.component.ts
View File

@@ -155,8 +155,16 @@ export class CodeComponent extends CellView implements OnInit, OnChanges {
if (!shouldConnect && connectionService && connectionService.isConnected(cellUri)) { if (!shouldConnect && connectionService && connectionService.isConnected(cellUri)) {
connectionService.disconnect(cellUri).catch(e => this.logService.error(e)); connectionService.disconnect(cellUri).catch(e => this.logService.error(e));
} else if (shouldConnect && this._model.context && this._model.context.id !== DEFAULT_OR_LOCAL_CONTEXT_ID) { } else if (shouldConnect && this._model.context && this._model.context.id !== DEFAULT_OR_LOCAL_CONTEXT_ID) {
// Don't connect immediately in case the user is switching cells quickly (such as holding down the arrow key to navigate through cells)
// Instead wait a small bit and then check if the cell is still active, and if it is at that point then connect so we aren't thrashing
// connections
setTimeout(() => {
if (this.isActive()) {
connectionService.connect(this._model.context, cellUri).catch(e => this.logService.error(e)); connectionService.connect(this._model.context, cellUri).catch(e => this.logService.error(e));
} }
}, 250);
}
} }
} }