Skip to content

[ES|QL] Make SubstituteSurrogateExpressions recognize SurrogateExpression attached to MultiTypeEsField #126645

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
fang-xing-esql opened this issue Apr 10, 2025 · 1 comment
Open

[ES|QL] Make SubstituteSurrogateExpressions recognize SurrogateExpression attached to MultiTypeEsField #126645

fang-xing-esql opened this issue Apr 10, 2025 · 1 comment
Assignees
@fang-xing-esql
Labels
:Analytics/ES|QL AKA ESQL >enhancement Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo)

Comments

@fang-xing-esql
Copy link
Member

#126532 Introduces surrogate expressions for to_ip (conversion)function. Traditionally, the substitute surrogate expressions is done by LogicalPlanOptimizer's substitutions batch. Because SubstituteSurrogateExpressions cannot recognize the conversion/surrogate functions attached to MultiTypeEsField today, the substitution of to_ip for multi-typed fields is done in Analyzer.

This issue is open to investigate the possibility to improve SubstituteSurrogateExpressions to recognize the conversion/surrogate functions attached to MultiTypeEsField, so that the substitutions can be centralized in LogicalPlanOptimizer
@fang-xing-esql fang-xing-esql self-assigned this Apr 10, 2025
@elasticsearchmachine elasticsearchmachine added the Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) label Apr 10, 2025
@elasticsearchmachine
Copy link
Collaborator

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

@fang-xing-esql fang-xing-esql mentioned this issue Apr 10, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@fang-xing-esql fang-xing-esql

Labels
:Analytics/ES|QL AKA ESQL >enhancement Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo)
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@elasticsearchmachine @fang-xing-esql