fix: remove hardcoded memsize system option (#1188)

sassoftware · Sep 5, 2024 · 94d666c · 94d666c
1 parent 4fa0b90
commit 94d666c
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 14 deletions.
diff --git a/client/src/connection/rest/context.ts b/client/src/connection/rest/context.ts
@@ -4,7 +4,13 @@ import { l10n } from "vscode";
 
 import axios, { AxiosResponse } from "axios";
 
-import { Context, ContextSummary, ContextsApi, Link } from "./api/compute";
+import {
+  Context,
+  ContextSummary,
+  ContextsApi,
+  Link,
+  SessionRequest,
+} from "./api/compute";
 import { BaseCompute, Compute, getApiConfig } from "./common";
 import { ComputeSession } from "./session";
 
@@ -73,15 +79,13 @@ export class ComputeContext extends Compute {
     }
 
     //Create the session
-    //TODO: Add session create options
-    //TODO: Session request should be an interface
-    const body = {
+    const body: SessionRequest = {
       version: 1,
       name: "mysess",
       description: "This is a session",
       attributes: {},
       environment: {
-        options: ["-validmemname EXTEND", "-validvarname ANY", "-memsize 0"],
+        options: ["-validmemname EXTEND", "-validvarname ANY"],
         autoExecLines: [],
       },
     };

diff --git a/client/src/connection/rest/server.ts b/client/src/connection/rest/server.ts
@@ -4,15 +4,11 @@ import { l10n } from "vscode";
 
 import axios, { AxiosResponse } from "axios";
 
-import { Link, Server, ServersApi } from "./api/compute";
+import { Link, Server, ServersApi, SessionRequest } from "./api/compute";
 import { BaseCompute, Compute, getApiConfig, stateOptions } from "./common";
 import { ComputeSession } from "./session";
 
-const DEFAULT_COMPUTE_OPTS = [
-  "-validmemname EXTEND",
-  "-validvarname ANY",
-  "-memsize 0",
-];
+const DEFAULT_COMPUTE_OPTS = ["-validmemname EXTEND", "-validvarname ANY"];
 
 export class ComputeServer extends Compute {
   api;
@@ -87,9 +83,7 @@ export class ComputeServer extends Compute {
     }
 
     //Create the session
-    //TODO: Add session create options
-    //TODO: Session request should be an interface
-    const body = {
+    const body: SessionRequest = {
       version: 1,
       name: "mysess",
       description: "This is a session",