cube-js/cube
GitHub で見るExtra brackets need to be specified when using dynamic measures
Open
#6,064 opened on 2023年1月26日
help wanted
Repository metrics
- Stars
- (19,563 stars)
- PR merge metrics
- (平均マージ 5d 16h) (30d で 138 merged PRs)
説明
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:
- someCoef sql should be
(${profit}) / ${sales}(profit inside brackets) - 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