featurebase/sql3
pokeeffe-molecula 158cc669d9
Tighten up ORDER BY (fb 507) (#2318)
* tighten up checks for order by expressions fixed ordering by expressions

* added testing to cover order by cases

* Add DecimalAgg member to proto GroupCount definition

In DAX, where we have split the orchestrator from the executor, and the
orchestrator can run on a different host, there are cases where
`GroupCount`s can travel over the wire via the Internal Client. In these
cases, when the group count contains a decimal aggregate, we need to
send that value as the appropriate type.

* fixed missing cases in order by and case block eval

Co-authored-by: Travis Turner <travis@molecula.com>
2022-11-30 11:32:37 -06:00
..
parser Fix formatting in CLI results with custom SQLResonse.UnmarshalJSON (#2305) 2022-11-21 18:43:53 -06:00
planner Tighten up ORDER BY (fb 507) (#2318) 2022-11-30 11:32:37 -06:00
test Tighten up ORDER BY (fb 507) (#2318) 2022-11-30 11:32:37 -06:00
errors.go Tighten up ORDER BY (fb 507) (#2318) 2022-11-30 11:32:37 -06:00
interfaces.go added /sql endpoint; implemented SHOW TABLES (#1935) 2022-08-31 13:01:05 -07:00
main_test.go use testhook test cleanup 2022-09-26 12:33:43 -05:00
sql_complex_test.go Fix formatting in CLI results with custom SQLResonse.UnmarshalJSON (#2305) 2022-11-21 18:43:53 -06:00
sql_test.go Fix formatting in CLI results with custom SQLResonse.UnmarshalJSON (#2305) 2022-11-21 18:43:53 -06:00