fix: remove redundant commands from redis script

jsdelivr · Dec 5, 2023 · e30da84 · e30da84
1 parent 2879a12
commit e30da84
Show file tree

Hide file tree

Showing 3 changed files with 74 additions and 41 deletions.
diff --git a/public/v1/spec.yaml b/public/v1/spec.yaml
@@ -47,7 +47,7 @@ paths:
 
         - Set the `inProgressUpdates` option to `true` if the application is running in interactive mode so that the user sees the results right away.
           - If the application is interactive by default but also implements a "CI" mode to be used in scripts, do not set the flag in the CI mode.
-        - To use the same probes in different measurements pass id of the first measurement to the `locations` option.
+        - To use the same probes in different measurements pass id of the initial measurement to the `locations` option.
       requestBody:
         content:
           application/json:

diff --git a/src/lib/redis/scripts.ts b/src/lib/redis/scripts.ts
@@ -78,10 +78,6 @@ export const recordResult: RecordResultScript = defineScript({
 		return
 	end
 
-	redis.call('HDEL', 'gp:in-progress', measurementId)
-	redis.call('DEL', awaitingKey)
-	redis.call('JSON.SET', key, '$.status', '"finished"')
-
 	return redis.call('JSON.GET', key)
 	`,
 	transformArguments (measurementId, testId, data) {