fix: TypeError: Cannot read property 'path' of undefined -- Case when partitioned originalSql is resolved for query without time dimension and incremental refreshKey is used

Author: paveltiunov

packages/cubejs-schema-compiler/adapter/BaseQuery.js

@@ -1760,11 +1760,15 @@ class BaseQuery {
             if (!preAggregation.partitionGranularity) {
               throw new UserError(`Incremental refresh key can only be used for partitioned pre-aggregations`);
             }
-            refreshKey = this.incrementalRefreshKey(
-              preAggregationQueryForSql,
-              refreshKey,
-              { window: preAggregation.refreshKey.updateWindow }
-            );
+            // TOOD Case when partitioned originalSql is resolved for query without time dimension.
+            // Consider fallback to not using such originalSql for consistency?
+            if (preAggregationQueryForSql.timeDimensions.length) {
+              refreshKey = this.incrementalRefreshKey(
+                preAggregationQueryForSql,
+                refreshKey,
+                { window: preAggregation.refreshKey.updateWindow }
+              );
+            }
           }
           if (preAggregation.refreshKey.every || preAggregation.refreshKey.incremental) {
             return {

packages/cubejs-server-core/core/index.js

@@ -73,6 +73,8 @@ const devLogger = (level) => (type, { error, warning, ...message }) => {
           .concat(R.takeLast(queryMaxLines / 2, formatted));
       }
       return `${prefix}\n--\n  ${formatted.join('\n')}\n--${showRestParams ? `\n${restParams}` : ''}`;
+    } else if (query) {
+      return `${prefix}\n--\n${JSON.stringify(query, null, 2)}\n--${showRestParams ? `\n${restParams}` : ''}`;
     }
     return `${prefix}${showRestParams ? `\n${restParams}` : ''}`;
   };