Skip to content

ES|QL: tests for FORK's evaluation of field names used in field_caps resolve calls #127208

@astefan

Description

@astefan

Description

Following the improvements made to FORK, it is necessary for the field names used in field_caps calls the IndexResolver is performing to be correctly tested and covered for fork as well.

These tests should go in IndexResolverFieldNamesTests.
Also, a comment here would be appreciated. The fieldNames method is trying to avoid asking for all fields from field_caps for performance reasons and it's trying to be smart in what fields it actually needs. It would help code maintainers and contributors to understand why fork needs all the fields in all cases.

Metadata

Metadata

Assignees

Labels

:Analytics/ES|QLAKA ESQL>testIssues or PRs that are addressing/adding testsTeam:AnalyticsMeta label for analytical engine team (ESQL/Aggs/Geo)

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions