Skip to content
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

SQL: Investigate the way text manipulating functions extract the value from the fields #31894

Open
astefan opened this issue Jul 9, 2018 · 4 comments
Open

SQL: Investigate the way text manipulating functions extract the value from the fields #31894

astefan opened this issue Jul 9, 2018 · 4 comments
Assignees
@astefan
Labels
:Analytics/SQL SQL querying Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) >tech debt

Comments

@astefan
Copy link
Contributor

astefan commented Jul 9, 2018

At the moment, if a text field is passed to a text manipulating function, we try to see if the fields has a keyword subfield and if it does, we use that one and extract the value from doc_values. Should we look into see if the fields has the source stored and extract that instead? (and pay the performance penalty of retrieving the source and extract it)
@astefan astefan added the :Analytics/SQL SQL querying label Jul 9, 2018
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-search-aggs

@astefan astefan self-assigned this Jul 9, 2018
@astefan
Copy link
Contributor Author

astefan commented Jul 9, 2018

Related discussion: #31874 (comment)

@nik9000
Copy link
Member

nik9000 commented Jul 9, 2018

Personally I think we should reject queries rather than pay the performance cost to load _source at search time. Loading it at fetch time is fine by me though.

@rjernst rjernst added the Team:QL (Deprecated) Meta label for query languages team label May 4, 2020
@wchaparro wchaparro removed the Team:QL (Deprecated) Meta label for query languages team label Jan 17, 2024
@elasticsearchmachine elasticsearchmachine added the Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) label Jan 17, 2024
@elasticsearchmachine
Copy link
Collaborator

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@astefan astefan

Labels
:Analytics/SQL SQL querying Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) >tech debt
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
6 participants
@nik9000 @rjernst @astefan @wchaparro @elasticmachine @elasticsearchmachine