cube-js/cube

Funnel step queries should be wrapped in braces

開放

#360 建立於 2020年1月27日

在 GitHub 查看貢獻者指南
 (3 則留言) (0 個反應) (0 位負責人)Rust (1,965 個分叉)batch import
help wanted

倉庫指標

星標
 (19,563 顆星)
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