fix: TypeError: Cannot read property 'evaluateSymbolSqlWithContext' o…

…f undefined
cube-js · Jan 17, 2020 · 125afd7 · 125afd7
1 parent 7df5bd6
commit 125afd7
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/packages/cubejs-schema-compiler/adapter/BaseQuery.js b/packages/cubejs-schema-compiler/adapter/BaseQuery.js
@@ -1325,8 +1325,11 @@ class BaseQuery {
       const preAggregationForQuery = this.preAggregations.findPreAggregationForQuery();
       if (preAggregationForQuery) {
         return {
-          renewalThreshold: this.renewalThreshold(!!preAggregationForQuery.refreshKey),
-          queries: this.preAggregationInvalidateKeyQueries(preAggregationForQuery.cube, preAggregationForQuery)
+          renewalThreshold: this.renewalThreshold(!!preAggregationForQuery.preAggregation.refreshKey),
+          queries: this.preAggregationInvalidateKeyQueries(
+            preAggregationForQuery.cube,
+            preAggregationForQuery.preAggregation
+          )
         };
       }
     }

diff --git a/packages/cubejs-schema-compiler/test/PreAggregationsTest.js b/packages/cubejs-schema-compiler/test/PreAggregationsTest.js
@@ -412,6 +412,8 @@ describe('PreAggregations', function test() {
       console.log(JSON.stringify(preAggregationsDescription, null, 2));
 
       preAggregationsDescription[0].invalidateKeyQueries[0][0].should.match(/>=/);
+      console.log(JSON.stringify(query.cacheKeyQueries().queries, null, 2));
+      query.cacheKeyQueries().queries[0][0].should.match(/max/);
 
       return dbRunner.testQueries(tempTablePreAggregations(preAggregationsDescription).concat([
         query.buildSqlAndParams()