fix: include maskedMembers in collectFrom cache key for correct join resolution

When a member is masked, evaluateSymbolSql takes a different path
(mask.sql instead of regular sql), which may reference different
cubes requiring different joins. The compilerCache was caching the
collection results without considering masking state, causing stale
join hints when the same member was first collected without masking
and then with masking.

Add 'masked' suffix to the cache key for masked members so that
masked and unmasked collection results are cached separately.

Co-authored-by: Pavel Tiunov <pavel.tiunov@gmail.com>

Author: cursoragent

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

@@ -2843,17 +2843,20 @@ export class BaseQuery {
    */
   collectFrom(membersToCollectFrom, fn, methodName, cache) {
     const methodCacheKey = Array.isArray(methodName) ? methodName : [methodName];
+    const maskedMembers = this.maskedMembers;
     return R.pipe(
       R.map(f => f.getMembers()),
       R.flatten,
-      R.map(s => (
-        (cache || this.compilerCache).cache(
+      R.map(s => {
+        const memberPath = s.path() ? s.path().join('.') : null;
+        const isMasked = memberPath && maskedMembers && maskedMembers.has(memberPath);
+        return (cache || this.compilerCache).cache(
           ['collectFrom'].concat(methodCacheKey).concat(
-            s.path() ? [s.path().join('.')] : [s.cube().name, s.expression?.toString() || s.expressionName || s.definition().sql]
-          ),
+            memberPath ? [memberPath] : [s.cube().name, s.expression?.toString() || s.expressionName || s.definition().sql]
+          ).concat(isMasked ? ['masked'] : []),
           () => fn(() => this.traverseSymbol(s))
-        )
-      )),
+        );
+      }),
       R.unnest,
       R.uniq,
       R.filter(R.identity)