breaking the loop

Author: KSDaemon

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

@@ -2241,6 +2241,13 @@ export class BaseQuery {
       }
     }
 
+    // This is a special recursion guard that might happen sometimes, like
+    // during alias members collection which invokes sql evaluation of all members
+    // when FILTER_PARAMS is proxied for SQL evaluation.
+    if (parentMember === memberPath && this.safeEvaluateSymbolContext().aliasGathering) {
+      return '';
+    }
+
     this.safeEvaluateSymbolContext().currentMember = memberPath;
     try {
       if (type === 'measure') {
@@ -3938,8 +3945,10 @@ export class BaseQuery {
     const query = this;
     return members.map(
       member => {
-        const collectedMembers = query
-          .collectFrom([member], query.collectMemberNamesFor.bind(query), 'collectMemberNamesFor');
+        const collectedMembers = query.evaluateSymbolSqlWithContext(
+          () => query.collectFrom([member], query.collectMemberNamesFor.bind(query), 'collectMemberNamesFor'),
+          { aliasGathering: true }
+        );
         const memberPath = member.expressionPath();
         let nonAliasSeen = false;
         return collectedMembers