cube-js/cube

Meta API (`/meta`): Add the `sql_table` field to the response

Open

#8711 aperta il 16 set 2024

Vedi su GitHub
 (3 commenti) (2 reazioni) (0 assegnatari)Rust (1965 fork)batch import
api:resthelp wanted

Metriche repository

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

Descrizione

The /v1/meta API endpoint does not include the sql_table field. Even with the ?extended query param the sql_table field does not exist, even though sql does. It would also be great to have this included unless there is a good reason not to.

I believe it's the following function that needs to be adapted along with the unit tests: https://github.com/cube-js/cube/blob/e8d81f2e04ce61215b1185d9adc4320233bc34da/packages/cubejs-api-gateway/src/helpers/transformMetaExtended.ts#L37

I'm happy to open a pull request for the same!

Guida contributor

Direzione di ricerca
Individua la funzione transformMetaExtended in packages/cubejs api gateway/src/helpers/transformMetaExtended.ts. Aggiungi il campo 'sql table' all'oggetto restituito, probabilmente accanto a 'sql'. Aggiorna i test unitari nella stessa directory per verificare che il nuovo campo sia incluso nella risposta. Assicurati che il campo sia presente sia con che senza il parametro di query 'extended'.
Tech stack
typescript
Dominio
backend
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
TypeScriptGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
85