-
Notifications
You must be signed in to change notification settings - Fork 25.7k
ESQL: Fix attribute only in full text function not found #137395
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.
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
ESQL: Fix attribute only in full text function not found #137395
Conversation
|
Pinging @elastic/es-analytical-engine (Team:Analytics) |
|
Hi @julian-elastic, I've created a changelog YAML for you. |
mouhc1ne
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
|
@mouhc1ne Can you please take another look? I had to add a capability to get the new UTs to pass with BWC tests. |
mouhc1ne
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
) A verification exception is thrown if there is LOOKUP JOIN with a FULL TEXT FUNCTION on the LOOKUP INDEX. The field from the LOOKUP INDEX is only used in the full text function. There is KEEP that selects some fields, but not the field in question. The fix involves making sure that all the fields from the JoinOnCondition of the LOOKUP JOIN are collected during resolveFieldNames() to be used for field caps. Resolves elastic#137396
A verification exception is thrown if there is LOOKUP JOIN with a FULL TEXT FUNCTION on the LOOKUP INDEX. The field from the LOOKUP INDEX is only used in the full text function. There is KEEP that selects some fields, but not the field in question.
The fix involves making sure that all the fields from the JoinOnCondition of the LOOKUP JOIN are collected during resolveFieldNames() to be used for field caps.
Resolves #137396