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.

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

[8.17] ESQL: connect_transport_exception should be thrown instead of verification_exception when ENRICH-ing if remote is disconnected (#119750) #119881

Conversation

pawankartik-elastic
Copy link
Contributor

Backports the following commits to 8.17:

…fication_exception` when ENRICH-ing if remote is disconnected (elastic#119750)

* fix: `verification_exception` is thrown instead of `connect_transport_exception` in ENRICH

In the context of ENRICH, if a remote is disconnected and skip
unavailable is set to `true`, then `verification_exception` is thrown
instead of `connect_transport_exception`. This PR fixes this and adds
the IT tests for ENRICH for RCS 1 and RCS 2.

* Update docs/changelog/119750.yaml

* Update 119750.yaml

---------

Co-authored-by: Michael Peterson <[email protected]>
@pawankartik-elastic pawankartik-elastic added :Search Foundations/Search Catch all for Search Foundations >bug auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport Team:Search Foundations Meta label for the Search Foundations team in Elasticsearch labels Jan 9, 2025
@elasticsearchmachine elasticsearchmachine merged commit c71c008 into elastic:8.17 Jan 9, 2025
15 checks passed
@pawankartik-elastic pawankartik-elastic deleted the backport/8.17/pr-119750 branch January 9, 2025 17:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport >bug :Search Foundations/Search Catch all for Search Foundations Team:Search Foundations Meta label for the Search Foundations team in Elasticsearch v8.17.2
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants