query_window -> query_window_timestamp
This commit is contained in:
parent
baa83363fa
commit
7127779182
1
TODO.md
1
TODO.md
@ -227,6 +227,7 @@ These are roughly in order of completition
|
||||
- [ ] instead of configuring each cache with MB sizes, have one value for total memory footprint and then percentages for each cache
|
||||
- [ ] proper authentication on rpc_key_id
|
||||
- we have bearer token auth for user_id, but rpc_key_id needs more code
|
||||
- [ ] fix tests
|
||||
|
||||
## V1
|
||||
|
||||
|
@ -193,10 +193,10 @@ pub fn filter_query_window_seconds(
|
||||
);
|
||||
|
||||
let q = q
|
||||
.column_as(expr, "query_window")
|
||||
.group_by(Expr::cust("query_window"))
|
||||
.column_as(expr, "query_window_timestamp")
|
||||
.group_by(Expr::cust("query_window_timestamp"))
|
||||
// TODO: is there a simpler way to order_by?
|
||||
.order_by_asc(SimpleExpr::Custom("query_window".to_string()));
|
||||
.order_by_asc(SimpleExpr::Custom("query_window_timestamp".to_string()));
|
||||
|
||||
Ok(q)
|
||||
}
|
||||
|
Loading…
Reference in New Issue
Block a user