-
Notifications
You must be signed in to change notification settings - Fork 195
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
Verify Indices Mappings #1067
Verify Indices Mappings #1067
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please see comments
@mickeyn I critically reviewed the code and changed it where it was beneficial to maintainability or operational coherence. |
are you still interested in this feature ? |
@mickeyn would you be able to take another look at this? |
@oalders yes |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Thanks @bodo-hugo-barwich and @mickeyn! |
This implements the deep comparison of the deployed mappings in the ElasticSearch cluster and the definitions in the project.
This will show automatic unexpected adjustments made by the ElasticSearch engine which do not correspond to the intended structure.
This feature was requested and defined at:
Fix Index Corruption
as a result of the blocking incident documented at:
MetaCPAN API - Indexing failed
It introduces some of the same functionality like:
Clear Unknown Indices
which is required to create meaningful tests.