chore: Reduce memory footprint for pre-aggregations with many partitions by caching partition SQL -- fix missing argument (#5941)

Author: paveltiunov

packages/cubejs-query-orchestrator/src/orchestrator/PreAggregations.ts

@@ -1450,7 +1450,7 @@ export class PreAggregationPartitionRangeLoader {
     this.requestId = options.requestId;
     this.lambdaQuery = options.lambdaQuery;
     this.dataSource = preAggregation.dataSource;
-    this.compilerCacheFn = options.compilerCacheFn || ((subKey, cacheFn) => cacheFn());
+    this.compilerCacheFn = options.compilerCacheFn || ((subKey, cacheFn) => { console.log(`!!!! cacheFn`, cacheFn); return cacheFn(); });
   }
 
   private async loadRangeQuery(rangeQuery: QueryTuple, partitionRange?: QueryDateRange) {

packages/cubejs-server-core/src/core/CompilerApi.js

@@ -148,7 +148,7 @@ export class CompilerApi {
     if (this.sqlCache) {
       return (subKey, cacheFn) => compilers.compilerCache.getQueryCache(key).cache(path.concat(subKey), cacheFn);
     } else {
-      return (cacheFn) => cacheFn();
+      return (subKey, cacheFn) => cacheFn();
     }
   }