Skip to content

Commit

Permalink
fix auto unnest check (#5539)
Browse files Browse the repository at this point in the history
  • Loading branch information
pjain1 committed Aug 28, 2024
1 parent 4cb4ce2 commit fe24333
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions runtime/metricsview/astexpr.go
Original file line number Diff line number Diff line change
Expand Up @@ -211,11 +211,11 @@ func (b *sqlExprBuilder) writeBinaryCondition(exprs []*Expression, op Operator)

// Generate unnest join
unnestTableAlias := b.ast.generateIdentifier()
unnestFrom, ok, err := b.ast.dialect.LateralUnnest(leftExpr, unnestTableAlias, left.Name)
unnestFrom, auto, err := b.ast.dialect.LateralUnnest(leftExpr, unnestTableAlias, left.Name)
if err != nil {
return err
}
if !ok {
if auto {
// Means the DB automatically unnests, so we can treat it as a normal value
return b.writeBinaryConditionInner(nil, right, leftExpr, op)
}
Expand Down

1 comment on commit fe24333

@github-actions
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎉 Published on https://ui.rilldata.com as production
🚀 Deployed on https://66cef75dba750830079f351a--rill-ui.netlify.app

Please sign in to comment.