Merge pull request #507 from metrico/fix/empty_cached_ids_bug

debug
metrico · May 31, 2024 · 78d5eb1 · 78d5eb1
2 parents f96a7e6 + aa1f3bf
commit 78d5eb1
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/traceql/clickhouse_transpiler/attr_condition.js b/traceql/clickhouse_transpiler/attr_condition.js
@@ -71,7 +71,7 @@ module.exports = class Builder {
       const having = self.getCond(self.conds)
       self.aggregator(sel)
       sel.conditions = Sql.And(sel.conditions, Sql.Or(...self.where))
-      if (Array.isArray(ctx.randomFilter) && Array.isArray(ctx.cachedTraceIds)) {
+      if (Array.isArray(ctx.randomFilter) && Array.isArray(ctx.cachedTraceIds) && ctx.cachedTraceIds.length > 0) {
         sel.conditions = Sql.And(
           sel.conditions,
           Sql.Or(