Merge pull request #108 from ThePrez/feat/numrowsupdated

Add update count to user message when no results are returned

Author: worksofliam

src/connection/serverComponent.ts

@@ -14,7 +14,7 @@ const octokit = new Octokit();
 
 // During development, you can set the SERVER_VERSION in .vscode/launch.json
 // Otherwise, fall back to the working version
-const SERVER_VERSION = process.env[`SERVER_VERSION`] || `v1.1.0`;
+const SERVER_VERSION = process.env[`SERVER_VERSION`] || `v1.2.0`;
 
 const ExecutablePathDir = `$HOME/.vscode/`;
 

src/connection/types.ts

@@ -42,6 +42,7 @@ export interface QueryResult<T> extends ServerResponse {
   metadata: QueryMetaData,
   is_done: boolean;
   has_results: boolean;
+  update_count: number;
   data: T[];
 }
 

src/views/results/html.ts

@@ -97,7 +97,7 @@ export function generateScroller(basicSelect: string, isCL: boolean): string {
 
                   const nextButton = document.getElementById("nextButton");
                   if (data.rows === undefined && totalRows === 0) {
-                    nextButton.innerText = 'Query executed with no result set returned.';
+                    nextButton.innerText = 'Query executed with no result set returned. Rows affected: ' + data.update_count;
                   } else {
                     nextButton.innerText = noMoreRows ? ('Loaded ' + totalRows + '. End of data') : ('Loaded ' + totalRows + '. Fetching more...');
                   }

src/views/results/index.ts

@@ -52,6 +52,7 @@ class ResultSetPanelProvider {
             rows: queryResults.data,
             columnList: queryResults.metadata ? queryResults.metadata.columns.map(x=>x.name) : undefined, // Query.fetchMore() doesn't return the metadata
             queryId: queryObject.getId(),
+            update_count: queryResults.update_count,
             isDone: queryResults.is_done
           });