[Discover] Handle text based (ES|QL) query checks consistently in Discover #181639
Labels
chore
Feature:Discover
Discover Application
impact:low
Addressing this issue will have a low level of impact on the quality/strength of our product.
loe:small
Small Level of Effort
Team:DataDiscovery
Discover App Team (Document Explorer, Saved Search, Surrounding documents, Graph)
Projects
Currently in Discover we use a few different methods of checking if a query is text based (ES|QL):
getRawRecordType(query) === RecordRawType.PLAIN
isTextBasedQuery(query)
isOfAggregateQueryType(query)
We also use different variable names for the result which is confusing:
isTextBased
isPlainRecord
We should align on a single method of checking for text based / ES|QL queries and use consistent naming. Possibly something including ES|QL for clarity, along these lines:
const isEsqlQuery = isEsqlQueryType(query)
.The text was updated successfully, but these errors were encountered: