georgia-tech-db / eva / b8449b3c-2617-4cb5-babf-0c08f37c4c66

80%

master: 0%

push

circle-ci

feat: add support for enabling ORDER BY on non projected columns (#1155)

In response to PR #1147, we've introduced support for the use of ORDER
BY on non-projected columns

```
SELECT name FROM meme_images
ORDER BY Similarity(SiftFeatureExtractor(Open("{}")), SiftFeatureExtractor(data))
LIMIT 1;

SELECT FaceDetector(data) FROM MyVideo WHERE id < 5 order by scores;

SELECT AirForecast(12) order by y;

Select name FROM meme_images ORDER BY name;
```

The following query won't work. 

`Select name AS n FROM meme_images ORDER BY n;`

Currently, we execute projection as the last operation. Only when a
function expression is present it is executed before projection,
allowing its use in ORDER BY. A complete fix will require us to redesign
the binder. Right now, we've decided to defer this task because there is
no urgency to support it.

Ref:
https://github.com/duckdb/duckdb/blob/52a47a6b3/src/planner/expression_binder/order_binder.cpp#L68

28 of 28 new or added lines in 3 files covered. (100.0%)

9607 of 11984 relevant lines covered (80.17%)

1.44 hits per line