Skip to content

Cherry-pick #19980 to 7.x: Do not compare err with custom type#19982

Merged
michalpristas merged 1 commit intoelastic:7.xfrom
michalpristas:backport_19980_7.x
Jul 16, 2020
Merged

Cherry-pick #19980 to 7.x: Do not compare err with custom type#19982
michalpristas merged 1 commit intoelastic:7.xfrom
michalpristas:backport_19980_7.x

Conversation

@michalpristas
Copy link
Copy Markdown
Contributor

Cherry-pick of PR #19980 to 7.x branch. Original message:

What does this PR do?

Fixes #19976 by using comparison from errors package instead of comparing custom types

Why is it important?

Avoid panics

Checklist

  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have made corresponding change to the default configuration files
  • I have added tests that prove my fix is effective or that my feature works
  • I have added an entry in CHANGELOG.next.asciidoc or CHANGELOG-developer.next.asciidoc.

cc @ph

@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/ingest-management (Team:Ingest Management)

@botelastic botelastic bot added needs_team Indicates that the issue/PR needs a Team:* label and removed needs_team Indicates that the issue/PR needs a Team:* label labels Jul 16, 2020
@michalpristas michalpristas merged commit 67f4d61 into elastic:7.x Jul 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport bug Ingest Management:beta1 Group issues for ingest management beta1 review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants