fix: Empty data partitioned pre-aggregations are incorrectly handled …

…-- value provided is not in a recognized RFC2822 or ISO format
cube-js · Oct 31, 2021 · 9f3acd5 · 9f3acd5
1 parent 8182824
commit 9f3acd5
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/packages/cubejs-backend-shared/src/time.ts b/packages/cubejs-backend-shared/src/time.ts
@@ -85,7 +85,11 @@ export const utcToLocalTimeZone = (timezone: string, timestampFormat: string, ti
 
 export const extractDate = (data: any): string => {
   data = JSON.parse(JSON.stringify(data));
-  return moment.tz(data[0] && data[0][Object.keys(data[0])[0]], 'UTC').utc().format(moment.HTML5_FMT.DATETIME_LOCAL_MS);
+  const value = data[0] && data[0][Object.keys(data[0])[0]];
+  if (!value) {
+    return value;
+  }
+  return moment.tz(value, 'UTC').utc().format(moment.HTML5_FMT.DATETIME_LOCAL_MS);
 };
 
 export const addSecondsToLocalTimestamp = (timestamp: string, timezone: string, seconds: number): Date => {

diff --git a/packages/cubejs-query-orchestrator/src/orchestrator/PreAggregations.ts b/packages/cubejs-query-orchestrator/src/orchestrator/PreAggregations.ts
@@ -1189,6 +1189,9 @@ export class PreAggregationPartitionRangeLoader {
 
   private async partitionRanges() {
     const buildRange = await this.loadBuildRange();
+    if (!buildRange[0] || !buildRange[1]) {
+      return [];
+    }
     let dateRange = PreAggregationPartitionRangeLoader.intersectDateRanges(
       buildRange,
       this.preAggregation.matchedTimeDimensionDateRange,