cube-js/cube

Add an endpoint to get final SQL query (not parameterized)

Aperta

#4932 aperta il 18 lug 2022

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

Metriche repository

Star
 (19.563 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

Is your feature request related to a problem? Please describe. The endpoint /cubejs-api/v1/sql returns a parameterized query which is of no use.

Describe the solution you'd like An endpoint that returns the final SQL request ready to be pasted in DB ide (to debug and test).

Describe alternatives you've considered Manually replace the ? with filters value but it is error prone and tiresome.

Additional context related to https://github.com/cube-js/cube.js/issues/4614

Guida contributor

Direzione di ricerca
La richiesta del problema è per un nuovo endpoint simile a `/cubejs api/v1/sql/raw` che restituisca la query SQL finale con i parametri sostituiti. Attualmente, l'endpoint `/sql` esistente restituisce una query parametrizzata. Per implementarlo, è necessario modificare il codice del server che gestisce la generazione SQL. Fare riferimento all'implementazione esistente nella directory `cubesql` o `server core`. L'issue correlato #4614 potrebbe fornire contesto. Dovrebbe essere aggiunto un nuovo gestore di route che accetti gli stessi parametri di `/sql` ma esegua la sostituzione dei parametri prima di restituire. Assicurarsi che la sostituzione gestisca tutti i casi limite come i filtri di lista.
Tech stack
javascripttypescriptnodejs
Dominio
backend
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
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
GitNode.js
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
50