Skip to content

Throw 4xx instead of 5xx for ESQL malformed query params#134879

Merged
pmpailis merged 9 commits intoelastic:mainfrom
pmpailis:fix/134618_throwing_4xx_instead_of_5xx_for_esql_malformed_async_query_params
Sep 23, 2025
Merged

Throw 4xx instead of 5xx for ESQL malformed query params#134879
pmpailis merged 9 commits intoelastic:mainfrom
pmpailis:fix/134618_throwing_4xx_instead_of_5xx_for_esql_malformed_async_query_params

Conversation

@pmpailis
Copy link
Contributor

When parsing the params field for an ESQL query, ensure that we only support arrays and throw a ParsingException otherwise, instead of an IllegalStateException.

Closes #134618

@pmpailis pmpailis added >bug :Search Relevance/ES|QL Search functionality in ES|QL labels Sep 17, 2025
@elasticsearchmachine elasticsearchmachine added v9.2.0 Team:Search Relevance Meta label for the Search Relevance team in Elasticsearch labels Sep 17, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-search-relevance (Team:Search Relevance)

@elasticsearchmachine
Copy link
Collaborator

Hi @pmpailis, I've created a changelog YAML for you.

Copy link
Member

@carlosdelest carlosdelest 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, thanks @pmpailis !

pmpailis and others added 6 commits September 17, 2025 17:45
…xx_instead_of_5xx_for_esql_malformed_async_query_params
…xx_instead_of_5xx_for_esql_malformed_async_query_params
…ed_async_query_params' of github.com:pmpailis/elasticsearch into fix/134618_throwing_4xx_instead_of_5xx_for_esql_malformed_async_query_params
@pmpailis pmpailis enabled auto-merge (squash) September 23, 2025 09:02
@pmpailis
Copy link
Contributor Author

run elasticsearch-ci/part-1

@pmpailis pmpailis merged commit c543631 into elastic:main Sep 23, 2025
34 checks passed
@pmpailis pmpailis deleted the fix/134618_throwing_4xx_instead_of_5xx_for_esql_malformed_async_query_params branch January 16, 2026 15:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

>bug :Search Relevance/ES|QL Search functionality in ES|QL Team:Search Relevance Meta label for the Search Relevance team in Elasticsearch v9.2.0

3 participants