cube-js/cube

Funnel step queries should be wrapped in braces

Open

#360 aperta il 27 gen 2020

Vedi su GitHub
 (3 commenti) (0 reazioni) (0 assegnatari)Rust (1965 fork)batch import
help wanted

Metriche repository

Star
 (19.563 star)
Metriche merge PR
 (Merge medio 5g 16h) (138 PR mergiate in 30 g)

Descrizione

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

Guida contributor

Direzione di ricerca
Indaga il codice di generazione delle query funnel, probabilmente nei file del query builder. Cerca dove le query SQL dei passi vengono concatenate nella query esterna. La correzione prevede di racchiudere ogni query di passo tra parentesi `()` per prevenire errori di sintassi SQL. Rivedi i test correlati per le query funnel per garantirne la correttezza.
Tech stack
rustsql
Dominio
backendanalytics
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitSQLRust
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70