Fix zipkin missing and number type column responses

CHANGELOG.md

@@ -2,6 +2,11 @@
 
 All notable changes to the "vscode-vespa" extension will be documented in this file.
 
+## [1.1.4]
+
+- Make sure you can run query if zipkin is NOT defined.
+- Fix bug with number type columns.
+
 ## [1.1.3]
 
 - Fix issues with connecting to single node cluster at Vespa cluster startup and deployment.

client/package.json

@@ -3,7 +3,7 @@
 	"description": "VSCode part of a language server",
 	"author": "Matti Pehrs",
 	"license": "MIT",
-	"version": "1.1.3",
+	"version": "1.1.4",
 	"publisher": "pehrs.com",	
 	"engines": {
 		"vscode": "^1.75.0"

client/src/YqlResultsPanel.ts

@@ -182,7 +182,7 @@ export class YqlResultsPanel {
 			for (const column in fieldNames) {
 				if (column in child.fields) {
 					const value: string = child.fields[column];
-					if (value.startsWith("http")) {
+					if (value !== undefined && typeof value === 'string' && value.startsWith("http")) {
 						result += `<td><a href="${value}">${value}</a></td>`;
 					} else {
 						result += `<td>${value}</td>`;

client/src/extension.ts

@@ -533,7 +533,7 @@ async function runYqlQuery(context: ExtensionContext, yql: string): Promise<void
 			response.json()
 				.then((data: any) => {
 					outputChannel.appendLine("Got Vespa query response!");
-					if (data.trace) {
+					if (data.trace && vespaConfig.zipkinEndpoint !== undefined && vespaConfig.zipkinEndpoint.length > 0) {
 						uploadTrace2Zipkin(data)
 							.then((zipkinLink: string) => {
 								YqlResultsPanel.createOrShow(context.extensionUri, yql, data, zipkinLink, queryTimestamp);
@@ -545,7 +545,7 @@ async function runYqlQuery(context: ExtensionContext, yql: string): Promise<void
 						YqlResultsPanel.createOrShow(context.extensionUri, yql, data, undefined, queryTimestamp);
 					}
 				}).catch(reason => {
-					showError(`Could not parse response from ${queryEndpoint}, ${reason}`);
+					showError(`runYqlQuery(): Could not parse response from ${queryEndpoint}, ${reason}`);
 				});
 		})
 		.catch(reason => {

client/testFixture/news-query.yql

@@ -0,0 +1,18 @@
+{
+    "yql": "select * from news where true",
+    "offset": 5,
+    "ranking": {
+        "matchPhase": {
+            "ascending": true,
+            "maxHits": 15
+        }
+    },
+    "presentation" : {
+        "bolding": false,
+        "format": "json"
+    },
+    "trace": {
+      "level": 5,
+      "timestamps": true
+    }
+}

package.json

@@ -2,7 +2,7 @@
   "name": "vscode-vespa",
   "displayName": "vscode-vespa",
   "description": "Vespa AI Extension",
-  "version": "1.1.3",
+  "version": "1.1.4",
   "publisher": "pehrs-com",
   "license": "SEE LICENSE IN LICENSE",
   "icon": "media/vespa.png",

server/package.json

@@ -1,7 +1,7 @@
 {
 	"name": "vscode-vespa-server",
 	"description": "Example implementation of a language server in node.",
-	"version": "1.1.3",
+	"version": "1.1.4",
 	"author": "Matti Pehrs",
 	"license": "MIT",
 	"engines": {