Commit 8ead12a
authored
* Support `chart` command in PPL (#4579)
* WIP: Make poc implementation for chart command
Signed-off-by: Yuanchun Shen <[email protected]>
* Support param useother and otherstr
Signed-off-by: Yuanchun Shen <[email protected]>
* Support usenull and nullstr (when both row split and col split present)
Signed-off-by: Yuanchun Shen <[email protected]>
* Append a final aggregation to merge OTHER categories
Signed-off-by: Yuanchun Shen <[email protected]>
* Handle common agg functions for OTHER category for timechart
Signed-off-by: Yuanchun Shen <[email protected]>
* Fix timechart IT
Signed-off-by: Yuanchun Shen <[email protected]>
* Sort earliest results with asc order
Signed-off-by: Yuanchun Shen <[email protected]>
* Support non-string fields as column split
Signed-off-by: Yuanchun Shen <[email protected]>
* Fix min/earliest order & fix non-accumulative agg for chart
Signed-off-by: Yuanchun Shen <[email protected]>
* Hint non-null in aggregateWithTrimming
Signed-off-by: Yuanchun Shen <[email protected]>
* Add integration tests for chart command
Signed-off-by: Yuanchun Shen <[email protected]>
* Add unit tests
Signed-off-by: Yuanchun Shen <[email protected]>
* Add doc for chart command
Signed-off-by: Yuanchun Shen <[email protected]>
* Prompt users that multiple agg is not supported
Signed-off-by: Yuanchun Shen <[email protected]>
* Add explain ITs
Signed-off-by: Yuanchun Shen <[email protected]>
* Remove unimplemented support for multiple aggregations in chart command
Signed-off-by: Yuanchun Shen <[email protected]>
* Add unit tests for chart command
Signed-off-by: Yuanchun Shen <[email protected]>
* Remove irrelevant yaml test
Signed-off-by: Yuanchun Shen <[email protected]>
* Tweak chart.rst
Signed-off-by: Yuanchun Shen <[email protected]>
* Swap the order of chart output to ensure metrics come last
Signed-off-by: Yuanchun Shen <[email protected]>
* Filter rows without col split when calculate grand total
Signed-off-by: Yuanchun Shen <[email protected]>
* Chores: tweak code order
Signed-off-by: Yuanchun Shen <[email protected]>
* Add anonymize test to chart command
Signed-off-by: Yuanchun Shen <[email protected]>
* Change grammart from limit=top 10 to limit=top10
Signed-off-by: Yuanchun Shen <[email protected]>
* Update chart doc
Signed-off-by: Yuanchun Shen <[email protected]>
* Rename __row_number__ for chart to _row_number_chart_
Signed-off-by: Yuanchun Shen <[email protected]>
* Sort by row and col splits on top of chart results
Signed-off-by: Yuanchun Shen <[email protected]>
* Ignore rows without a row split in chart command
Signed-off-by: Yuanchun Shen <[email protected]>
* Keep categories with max summed values when top k is set
Signed-off-by: Yuanchun Shen <[email protected]>
* Simplify toAddHintsOnAggregate condition
Signed-off-by: Yuanchun Shen <[email protected]>
* Chores: eliminate unnecessary variables
Signed-off-by: Yuanchun Shen <[email protected]>
* Apply a non-null filter on fields referred by aggregations
Signed-off-by: Yuanchun Shen <[email protected]>
* Fix chart plans
Signed-off-by: Yuanchun Shen <[email protected]>
* Get rid of record class
Signed-off-by: Yuanchun Shen <[email protected]>
* Move ranking by column split to a helper function
Signed-off-by: Yuanchun Shen <[email protected]>
---------
Signed-off-by: Yuanchun Shen <[email protected]>
(cherry picked from commit 5523932)
* Downgrade language level to java 11
Signed-off-by: Yuanchun Shen <[email protected]>
* Fix explain IT
Signed-off-by: Yuanchun Shen <[email protected]>
* Take special handling for chart with use other explain IT
Signed-off-by: Yuanchun Shen <[email protected]>
---------
Signed-off-by: Yuanchun Shen <[email protected]>
1 parent 2b95c41 commit 8ead12a
File tree
44 files changed
+2347
-180
lines changed- core/src/main/java/org/opensearch/sql
- analysis
- ast
- tree
- calcite
- utils
- docs
- user/ppl
- cmd
- integ-test/src/test
- java/org/opensearch/sql/calcite/remote
- resources/expectedOutput
- calcite_no_pushdown
- calcite
- ppl/src
- main
- antlr
- java/org/opensearch/sql/ppl
- parser
- utils
- test/java/org/opensearch/sql/ppl
- calcite
- parser
- utils
Some content is hidden
Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.
44 files changed
+2347
-180
lines changedLines changed: 6 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
61 | 61 | | |
62 | 62 | | |
63 | 63 | | |
| 64 | + | |
64 | 65 | | |
65 | 66 | | |
66 | 67 | | |
| |||
768 | 769 | | |
769 | 770 | | |
770 | 771 | | |
| 772 | + | |
| 773 | + | |
| 774 | + | |
| 775 | + | |
| 776 | + | |
771 | 777 | | |
772 | 778 | | |
773 | 779 | | |
| |||
Lines changed: 5 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
49 | 49 | | |
50 | 50 | | |
51 | 51 | | |
| 52 | + | |
52 | 53 | | |
53 | 54 | | |
54 | 55 | | |
| |||
276 | 277 | | |
277 | 278 | | |
278 | 279 | | |
| 280 | + | |
| 281 | + | |
| 282 | + | |
| 283 | + | |
279 | 284 | | |
280 | 285 | | |
281 | 286 | | |
| |||
Lines changed: 55 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
| 20 | + | |
| 21 | + | |
| 22 | + | |
| 23 | + | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
| 27 | + | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
| 31 | + | |
| 32 | + | |
| 33 | + | |
| 34 | + | |
| 35 | + | |
| 36 | + | |
| 37 | + | |
| 38 | + | |
| 39 | + | |
| 40 | + | |
| 41 | + | |
| 42 | + | |
| 43 | + | |
| 44 | + | |
| 45 | + | |
| 46 | + | |
| 47 | + | |
| 48 | + | |
| 49 | + | |
| 50 | + | |
| 51 | + | |
| 52 | + | |
| 53 | + | |
| 54 | + | |
| 55 | + | |
0 commit comments