Skip to content

Correct exception type for rare path in dynamic mappings test#95443

Merged
romseygeek merged 1 commit intoelastic:mainfrom
romseygeek:bug/dynamic-mappings-exception-type
Apr 21, 2023
Merged

Correct exception type for rare path in dynamic mappings test#95443
romseygeek merged 1 commit intoelastic:mainfrom
romseygeek:bug/dynamic-mappings-exception-type

Conversation

@romseygeek
Copy link
Copy Markdown
Contributor

This was incorrectly changed in #92646 but never caught as it's a super
rare path for the test to take.

Fixes #95047

@romseygeek romseygeek added >test Issues or PRs that are addressing/adding tests :Search Foundations/Mapping Index mappings, including merging and defining field types v8.8.0 labels Apr 21, 2023
@romseygeek romseygeek requested review from javanna and quux00 April 21, 2023 10:16
@romseygeek romseygeek self-assigned this Apr 21, 2023
@elasticsearchmachine elasticsearchmachine added the Team:Search Meta label for search team label Apr 21, 2023
@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

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

@quux00
Copy link
Copy Markdown
Contributor

quux00 commented Apr 21, 2023

LGTM

@romseygeek romseygeek merged commit 9313830 into elastic:main Apr 21, 2023
@romseygeek romseygeek deleted the bug/dynamic-mappings-exception-type branch April 21, 2023 13:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Search Foundations/Mapping Index mappings, including merging and defining field types Team:Search Meta label for search team >test Issues or PRs that are addressing/adding tests v8.8.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CI] DynamicMappingIT testConflictingDynamicMappings failing

3 participants