cube-js/cube

Funnel step queries should be wrapped in braces

Open

#360 geöffnet am 27. Jan. 2020

Auf GitHub ansehen
 (3 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Rust (1.965 Forks)batch import
help wanted

Repository-Metriken

Stars
 (19.563 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 5T 16h) (138 gemergte PRs in 30 T)

Beschreibung

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

Research-Richtung
Untersuchen Sie den Code zur Erzeugung von Trichterabfragen, wahrscheinlich in den Query Builder Dateien. Suchen Sie nach der Stelle, an der Schritt SQL Abfragen in die äußere Abfrage verkettet werden. Die Korrektur besteht darin, jede Schrittabfrage in Klammern `()` zu setzen, um SQL Syntaxfehler zu vermeiden. Überprüfen Sie die zugehörigen Tests für Trichterabfragen, um die Korrektheit sicherzustellen.
Tech Stack
rustsql
Domain
backendanalytics
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitSQLRust
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70