cube-js / cube-js/cube

Funnel step queries should be wrapped in braces

Open
#360 3 comments 0 reactions 0 assignees View on GitHub
help wanted
Dominant language
Rust
Stars
20.8k
Forks
2.1k
Avg merge
1d 2h
Merged PRs (30d)
181

Description

**Describe the bug**
Currently some step queries can cause the generated SQL to fail.

**To Reproduce**
Steps to reproduce the behavior:
1. Create a new funnel
2. Add a large SQL query to a step
3. Attempt to run a query against the funnel cube
4. See error

**Expected behavior**
CubeJS should wrap any step SQL queries with braces `()` to prevent issues with the outer query.

**Version:**
v0.15.0

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.