cube-js/cube

Extra brackets need to be specified when using dynamic measures

Ouverte

#6 064 ouverte le 26 janv. 2023

 (2 commentaires) (0 réaction) (0 personne assignée)Rust (1 965 forks)batch import
help wanted

Métriques du dépôt

Stars
 (19 563 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

Is your feature request related to a problem? Please describe. Example of schema:

"sales":{
   "sql":"sum(${CUBE}."Sales")"
},
"wholesaleCost":{
   "sql":"sum(${CUBE}."WholesaleCost")"
},
"profit":{
   "sql":"${sales} - ${wholesaleCost}"
},
"someCoef":{
   "sql":"${profit} / ${sales}"
}

This schema is not working. someCoef generates wrong formula${sales} - ${wholesaleCost}/${sales} while correct is (${sales} - ${wholesaleCost})/${sales} It can be fixed in 2 ways:

  1. someCoef sql should be (${profit}) / ${sales} (profit inside brackets)
  2. profit sql should be (${sales} - ${wholesaleCost}) (whole formula inside brackets)

Describe the solution you'd like Cube should check and automatically add brackets if needed

Guide contributeur