cube-js/cube

Funnel step queries should be wrapped in braces

Open

#360 创建于 2020年1月27日

在 GitHub 查看
 (3 评论) (0 反应) (0 负责人)Rust (1,965 fork)batch import
help wanted

仓库指标

Star
 (19,563 star)
PR 合并指标
 (平均合并 5天 16小时) (30 天内合并 138 个 PR)

描述

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

贡献者指南

研究方向
调查漏斗查询生成代码，很可能在查询构建器文件中。查找将步骤 SQL 查询连接到外部查询的位置。修复方法是将每个步骤查询用括号 `()` 包裹起来，以防止 SQL 语法错误。审查漏斗查询的相关测试以确保正确性。
技术栈
rustsql
领域
backendanalytics
议题类型
缺陷
难度面向新贡献者的预计实现难度，1 表示很小改动，5 表示专家级工作。
2
预计时间有经验贡献者完成调查、实现、测试并准备 pull request 的粗略时间范围。
1-3 小时
活动状态议题当前的可参与程度：新鲜、活跃、陈旧、阻塞或等待维护者输入。
活跃
清晰度议题是否清楚说明期望改动、验收标准和下一步。
清晰
前置要求
GitSQLRust
新手友好度1-100 的估计分数，表示该议题对首次贡献者的友好程度。
70