Skip to content

Commit dc87e83

Browse files
committed
Changed so the execution time only runs on the first execute call
1 parent 2577335 commit dc87e83

File tree

2 files changed

+17
-4
lines changed

2 files changed

+17
-4
lines changed

src/views/results/html.ts

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -379,7 +379,12 @@ export function generateScroller(basicSelect: string, isCL: boolean, withCancel?
379379
if (data.rows === undefined && totalRows === 0) {
380380
document.getElementById(messageSpanId).innerText = 'Statement executed with no result set returned. Rows affected: ' + data.update_count;
381381
} else {
382-
document.getElementById(statusId).innerText = (noMoreRows ? ('Loaded ' + totalRows + ' rows in ' + data.executionTime + 'ms. End of data.') : ('Loaded ' + totalRows + ' rows in ' + data.executionTime + 'ms. More available.')) + ' ' + (updateTable ? 'Updatable.' : '');
382+
if (data.executionTime > 0) {
383+
document.getElementById(statusId).innerText = (noMoreRows ? ('Loaded ' + totalRows + ' rows in ' + data.executionTime + 'ms. End of data.') : ('Loaded ' + totalRows + ' rows in ' + data.executionTime + 'ms. More available.')) + ' ' + (updateTable ? 'Updatable.' : '');
384+
}
385+
else {
386+
document.getElementById(statusId).innerText = (noMoreRows ? ('Loaded ' + totalRows + ' rows. End of data.') : ('Loaded ' + totalRows + ' rows. More available.')) + ' ' + (updateTable ? 'Updatable.' : '');
387+
}
383388
document.getElementById(jobId).innerText = data.jobId ? data.jobId : '';
384389
document.getElementById(messageSpanId).style.visibility = "hidden";
385390
}

src/views/results/resultSetPanelProvider.ts

Lines changed: 11 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -99,9 +99,17 @@ export class ResultSetPanelProvider implements WebviewViewProvider {
9999

100100
if (this.currentQuery.getState() !== "RUN_DONE") {
101101
setCancelButtonVisibility(true);
102-
const startTime = performance.now();
103-
let queryResults = this.currentQuery.getState() == "RUN_MORE_DATA_AVAILABLE" ? await this.currentQuery.fetchMore() : await this.currentQuery.execute();
104-
const endTime = performance.now();
102+
let queryResults = undefined;
103+
let startTime = 0;
104+
let endTime = 0;
105+
if (this.currentQuery.getState() == "RUN_MORE_DATA_AVAILABLE") {
106+
queryResults = await this.currentQuery.fetchMore();
107+
}
108+
else {
109+
startTime = performance.now();
110+
queryResults = await this.currentQuery.execute();
111+
endTime = performance.now();
112+
}
105113
const jobId = this.currentQuery.getHostJob().id;
106114

107115
this._view.webview.postMessage({

0 commit comments

Comments
 (0)