You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Copy file name to clipboardExpand all lines: packages/cubejs-schema-compiler/src/adapter/BaseQuery.js
+11-6Lines changed: 11 additions & 6 deletions
Original file line number
Diff line number
Diff line change
@@ -227,7 +227,7 @@ export class BaseQuery {
227
227
dimensions: this.options.dimensions,
228
228
timeDimensions: this.options.timeDimensions,
229
229
filters: this.options.filters,
230
-
segments: this.options.segments,
230
+
egments: this.options.segments,
231
231
order: this.options.order,
232
232
contextSymbols: this.options.contextSymbols,
233
233
timezone: this.options.timezone,
@@ -307,12 +307,17 @@ export class BaseQuery {
307
307
}
308
308
309
309
prebuildJoin(){
310
-
/* if (!this.useNativeSqlPlanner) { We still need this join for the follback to preaggregation to work properly. This condition should be returned after the tesseract starts working with pre-aggregations
310
+
if(this.useNativeSqlPlanner){
311
311
// Tesseract doesn't require join to be prebuilt and there's a case where single join can't be built for multi-fact query
312
+
// But we need this join for a fallback when using pre-aggregations. So we’ll try to obtain the join but ignore any errors (which may occur if the query is a multi-fact one).
0 commit comments