fix: make cache optional fields optional (#9942)

This patch allows the getFromCache caller to only pass in
identifier or query field, as getFromCache already handles
the identifier and query field checks

Co-authored-by: ru.c <ru@shuffle.com>

src/cache/DbQueryResultCache.ts

@@ -131,7 +131,7 @@ export class DbQueryResultCache implements QueryResultCache {
     }
 
     /**
-     * Caches given query result.
+     * Get data from cache.
      * Returns cache result if found.
      * Returns undefined if result is not cached.
      */

src/cache/QueryResultCacheOptions.ts

@@ -6,7 +6,7 @@ export interface QueryResultCacheOptions {
      * Cache identifier set by user.
      * Can be empty.
      */
-    identifier: string
+    identifier?: string
 
     /**
      * Time, when cache was created.
@@ -21,7 +21,7 @@ export interface QueryResultCacheOptions {
     /**
      * Cached query.
      */
-    query: string
+    query?: string
 
     /**
      * Query result that will be cached.

src/cache/RedisQueryResultCache.ts

@@ -123,7 +123,7 @@ export class RedisQueryResultCache implements QueryResultCache {
     async synchronize(queryRunner: QueryRunner): Promise<void> {}
 
     /**
-     * Caches given query result.
+     * Get data from cache.
      * Returns cache result if found.
      * Returns undefined if result is not cached.
      */