Skip to content

Conversation

@nik9000
Copy link
Member

@nik9000 nik9000 commented Jul 23, 2025

Adds docs to UnresolvedAttribute and UnresolvedFunction explaining that we build them from tree-walking and resolve them at analyze time.

Adds docs to `UnresolvedAttribute` and `UnresolvedFunction` explaining
that we build them from tree-walking and resolve them at analyze time.
@elasticsearchmachine elasticsearchmachine added the Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) label Jul 23, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-analytical-engine (Team:Analytics)

* tree and then resolve them into other {@link Attribute} subclasses during
* analysis.
* <p>
* For example, ff they reference the data directly from lucene they'll be
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

if

Copy link
Contributor

@julian-elastic julian-elastic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, I found just one small typo

@nik9000 nik9000 enabled auto-merge (squash) July 23, 2025 22:18
@nik9000 nik9000 merged commit c4558ec into elastic:main Jul 23, 2025
33 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Analytics/ES|QL AKA ESQL >non-issue Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) v9.2.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants