search

elastic / kibana

Your window into the Elastic Stack
https://www.elastic.co/products/kibana
Other
19.35k stars 7.98k forks source link

[ES|QL] More usages of getESQLQueryColumns #182691

Closed stratoula closed 1 week ago

stratoula commented 1 week ago

Summary

This PR is an attempt to use the getESQLQueryColumns util everywhere we are trying to fetch the columns (using the limit 0).

elasticmachine commented 1 week ago

Pinging @elastic/kibana-esql (Team:ESQL)

stratoula commented 1 week ago

/ci

kibana-ci commented 1 week ago

:green_heart: Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
securitySolution 5496 5468 -28

Public APIs missing comments

Total count of every public API that lacks a comment. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats comments for more detailed information.

id before after diff
@kbn/esql-utils 34 41 +7

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
lens 1.4MB 1.4MB +167.0B
maps 3.0MB 3.0MB +64.0B
securitySolution 15.1MB 15.1MB +901.0B
stackAlerts 82.3KB 86.2KB +3.9KB
total +5.0KB

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
maps 51.2KB 51.5KB +342.0B
Unknown metric groups #### API count | id | [before](https://github.com/elastic/kibana/commit/71e7f5ae5586d97934dd9560d2b094db05ed2156) | [after](https://github.com/elastic/kibana/commit/d927107f7c7430cfc669edc2bc5c71b0aa617a86) | diff | | --- | --- | --- | --- | | `@kbn/esql-utils` | 36 | 43 | +7 |

History

To update your PR or re-run it, just comment with: @elasticmachine merge upstream