add prefix to keys for cache lookup (#1477)

* add prefix to keys for cache lookup * version bump
CarnegieLearningWeb · Apr 26, 2024 · e79c02b · e79c02b
1 parent ae8d8b2
commit e79c02b
Show file tree

Hide file tree

Showing 16 changed files with 24 additions and 23 deletions.
diff --git a/backend/package-lock.json b/backend/package-lock.json
diff --git a/backend/package.json b/backend/package.json
@@ -1,6 +1,6 @@
 {
     "name": "ab_testing_backend",
-    "version": "5.1.10",
+    "version": "5.1.11",
     "description": "Backend for A/B Testing Project",
     "scripts": {
         "install:all": "npm ci && cd packages/Scheduler && npm ci && cd ../Upgrade && npm ci",

diff --git a/backend/packages/Scheduler/package-lock.json b/backend/packages/Scheduler/package-lock.json
diff --git a/backend/packages/Scheduler/package.json b/backend/packages/Scheduler/package.json
@@ -1,6 +1,6 @@
 {
   "name": "ppl-upgrade-serverless",
-  "version": "5.1.10",
+  "version": "5.1.11",
   "description": "Serverless webpack example using Typescript",
   "main": "handler.js",
   "scripts": {

diff --git a/backend/packages/Upgrade/package-lock.json b/backend/packages/Upgrade/package-lock.json
diff --git a/backend/packages/Upgrade/package.json b/backend/packages/Upgrade/package.json
@@ -1,6 +1,6 @@
 {
     "name": "ab_testing_backend",
-    "version": "5.1.10",
+    "version": "5.1.11",
     "description": "Backend for A/B Testing Project",
     "main": "index.js",
     "scripts": {

diff --git a/backend/packages/Upgrade/src/api/services/CacheService.ts b/backend/packages/Upgrade/src/api/services/CacheService.ts
@@ -53,7 +53,8 @@ export class CacheService {
   }
 
   public async wrapFunction<T>(prefix: CACHE_PREFIX, keys: string[], functionToCall: () => Promise<T[]>): Promise<T[]> {
-    const cachedData = this.memoryCache ? await this.memoryCache.store.mget(...keys) : [];
+    const keysWithPrefix = keys.map((key) => prefix + key);
+    const cachedData = this.memoryCache ? await this.memoryCache.store.mget(...keysWithPrefix) : [];
 
     const allCachedFound = cachedData.every((cached) => !!cached);
     if (allCachedFound && env.caching.enabled) {

diff --git a/clientlibs/java/pom.xml b/clientlibs/java/pom.xml
@@ -9,7 +9,7 @@
 		at the same time that happen to rev to the same new version will be caught 
 		by a merge conflict. -->
 
-	<version>5.1.10</version>
+	<version>5.1.11</version>
 	<build>
 		<plugins>
 			<plugin>

diff --git a/clientlibs/js/package-lock.json b/clientlibs/js/package-lock.json
diff --git a/clientlibs/js/package.json b/clientlibs/js/package.json
@@ -1,6 +1,6 @@
 {
   "name": "upgrade_client_lib",
-  "version": "5.1.10",
+  "version": "5.1.11",
   "description": "Client library to communicate with the Upgrade server",
   "files": [
     "dist/*"

diff --git a/frontend/package-lock.json b/frontend/package-lock.json
diff --git a/frontend/package.json b/frontend/package.json
@@ -1,6 +1,6 @@
 {
   "name": "ab-testing",
-  "version": "5.1.10",
+  "version": "5.1.11",
   "license": "MIT",
   "scripts": {
     "ng": "ng",

diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "UpGrade",
-  "version": "5.1.10",
+  "version": "5.1.11",
   "description": "This is a combined repository for UpGrade, an open-source platform to support large-scale A/B testing in educational applications.  Learn more at www.upgradeplatform.org",
   "main": "index.js",
   "devDependencies": {

diff --git a/types/package-lock.json b/types/package-lock.json
diff --git a/types/package.json b/types/package.json
@@ -1,6 +1,6 @@
 {
   "name": "upgrade_types",
-  "version": "5.1.10",
+  "version": "5.1.11",
   "description": "",
   "main": "src/index.ts",
   "types": "src/index.ts",