ESQL: Enable multiple INLINESTATS + remaining tests

- [x] Let's get rid of this big TODO [here](https://github.com/elastic/elasticsearch/blob/c24f77f5471ea467f07ef797fb1647c361e20155/x-pack/plugin/esql/qa/testFixtures/src/main/resources/inlinestats.csv-spec#L2) :)
- [x] Especially important to enable multiple INLINESTATS in the same command (c.f. test case [here](https://github.com/elastic/elasticsearch/blob/dc8c35d91aab419bdfefa270c49c924ad8e0b4e9/x-pack/plugin/esql/qa/testFixtures/src/main/resources/inlinestats.csv-spec#L191-L200) and comment on why this may be broken [here](https://github.com/elastic/elasticsearch/pull/124335#discussion_r1985188283)).
 https://github.com/elastic/elasticsearch/pull/128917
- [x] INLINESTATS after SORT, see initial bug report [here](https://github.com/elastic/elasticsearch/issues/113727). This currently is nicely handled with an error message `Unbounded sort not supported yet [SORT gb DESC] please add a limit`
- [x] [INLINESTATS after STATS](https://github.com/elastic/elasticsearch/issues/124654)
 Tests added [here](https://github.com/elastic/elasticsearch/pull/128917/files#diff-a5ed793b796c14ec1f92807242ff2376e16888c8cf7627a4f6a8f57f23ed843aR1193-R1226)
- [x] Let's also re-enable this optimizer test and other tests disabled in https://github.com/elastic/elasticsearch/pull/115813, like this logical plan optimizer test: https://github.com/elastic/elasticsearch/blob/main/x-pack/plugin/esql/src/test/java/org/elasticsearch/xpack/esql/optimizer/LogicalPlanOptimizerTests.java
 Reenabled in https://github.com/elastic/elasticsearch/pull/132200
- [x] This query used to fail: `FROM kibana_sample_data_logs
| EVAL timestamp=DATE_TRUNC(5 minute, @timestamp) | INLINESTATS results = count(*) by timestamp | keep results, timestamp` Let's add a test and potentially fix it. 
   Fixed in: https://github.com/elastic/elasticsearch/pull/131204
- [x] Fix `FROM idx | EVAL ip = to_ip(host), x = to_string(host), y = to_string(host) | INLINESTATS max(id)` and re-enable the corresponding telemetry test case [muted here](https://github.com/elastic/elasticsearch/commit/681f5096e24967fc99699e682906776c1a75d39e). 
  Reenabled in https://github.com/elastic/elasticsearch/pull/132197

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

ESQL: Enable multiple INLINESTATS + remaining tests #124715

Metadata

Assignees

Labels

Type

Fields

Projects

Milestone

Relationships

Development

ESQL: Enable multiple INLINESTATS + remaining tests #124715

Description

Metadata

Metadata

Assignees

Labels

Type

Fields

Projects

Milestone

Relationships

Development

Issue actions