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

[8.10] Show concrete error when enrich index not exist rather than NPE (#99604) #100107

Merged
merged 1 commit into from Sep 29, 2023
Merged

[8.10] Show concrete error when enrich index not exist rather than NPE (#99604) #100107

merged 1 commit into from Sep 29, 2023

Conversation

jbaiera
Copy link
Member

@jbaiera jbaiera commented Sep 29, 2023

Backports the following commits to 8.10:

@puppylpg @jbaiera @elasticmachine
Show concrete error when enrich index not exist rather than NPE (elas…
4968776 
…tic#99604)

There should be NullPointerException check and throw index not found exception to the response 
so the user can understand what happens with the enrich index

---------

Co-authored-by: James Baiera <james.baiera@gmail.com>
Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
@jbaiera jbaiera added :Data Management/Ingest Node Execution or management of Ingest Pipelines including GeoIP >enhancement auto-merge Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport external-contributor Pull request authored by a developer outside the Elasticsearch team Team:Data Management Meta label for data/management team labels Sep 29, 2023
@elasticsearchmachine elasticsearchmachine mentioned this pull request Sep 29, 2023
Merged
@elasticsearchmachine elasticsearchmachine merged commit 4ac2db5 into elastic:8.10 Sep 29, 2023
12 checks passed
@jbaiera jbaiera deleted the backport/8.10/pr-99604 branch September 29, 2023 22:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
auto-merge Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport :Data Management/Ingest Node Execution or management of Ingest Pipelines including GeoIP >enhancement external-contributor Pull request authored by a developer outside the Elasticsearch team Team:Data Management Meta label for data/management team v8.10.3
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@jbaiera @elasticsearchmachine @puppylpg