From 17911278e862dc2b792097b13d22759abc0438fe Mon Sep 17 00:00:00 2001 From: Konstantin Burkalev <KSDaemon@gmail.com> Date: Thu, 23 Jan 2025 14:47:53 +0200 Subject: [PATCH] =?UTF-8?q?Add=20guard=20for=C2=A0time=20series=20without?= =?UTF-8?q?=20daterange?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- packages/cubejs-schema-compiler/src/adapter/BaseQuery.js | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/packages/cubejs-schema-compiler/src/adapter/BaseQuery.js b/packages/cubejs-schema-compiler/src/adapter/BaseQuery.js index cc63e3a21c708..f481102adcc3b 100644 --- a/packages/cubejs-schema-compiler/src/adapter/BaseQuery.js +++ b/packages/cubejs-schema-compiler/src/adapter/BaseQuery.js @@ -1429,6 +1429,10 @@ export class BaseQuery { return baseQueryFn(cumulativeMeasures, filters, false); } + if (this.timeDimensions.filter(d => !d.dateRange && d.granularity).length > 0) { + throw new UserError('Time series queries without dateRange aren\'t supported'); + } + // We can't do meaningful query if few time dimensions with different ranges passed, // it won't be possible to join them together without loosing some rows. const rangedTimeDimensions = this.timeDimensions.filter(d => d.dateRange && d.granularity);